Skip to content

RequestInfo should not include the access token in case it gets logge… #53

RequestInfo should not include the access token in case it gets logge…

RequestInfo should not include the access token in case it gets logge… #53

Triggered via push February 22, 2024 04:14
Status Failure
Total duration 8m 53s
Artifacts 2
Unit tests (.NET Framework)
4m 47s
Unit tests (.NET Framework)
Unit tests
0s
Unit tests
Publish prerelease packages
0s
Publish prerelease packages
Fit to window
Zoom out
Zoom in

Annotations

1 error and 12 warnings
Unit tests (.NET Framework)
Process completed with exit code 1.
Build
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-dotnet@v2, actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Build: ShopifySharp/Utilities/ShopifyOauthUtility.cs#L194
Possible null reference argument for parameter 'value' in 'KeyValuePair<string, string>.KeyValuePair(string key, string value)'.
Build: ShopifySharp/Utilities/ShopifyOauthUtility.cs#L248
Possible null reference argument for parameter 'accessToken' in 'AuthorizationResult.AuthorizationResult(string accessToken, string[]? grantedScopes)'.
Build: ShopifySharp/Utilities/ShopifyOauthUtility.cs#L292
Possible null reference argument for parameter 'accessToken' in 'AuthorizationResult.AuthorizationResult(string accessToken, string[]? grantedScopes)'.
Build: ShopifySharp/Infrastructure/ShopifyException.cs#L29
Non-nullable property 'RawBody' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
Build: ShopifySharp/Infrastructure/RequestResult.cs#L44
'RequestResult<T>.Response' is obsolete: 'This property is obsolete and will be removed in a future version of ShopifySharp. If you need to use the response headers, please use the ResponseHeaders property instead.'
Build: ShopifySharp/Infrastructure/ShopifyException.cs#L31
Non-nullable property 'RawBody' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
Build: ShopifySharp/Infrastructure/ShopifyException.cs#L41
'ShopifyException.HttpStatusCode' is obsolete: 'This property has been moved to the ShopifyHttpException and will be removed in a future version of ShopifySharp.'
Build: ShopifySharp/Infrastructure/ShopifyException.cs#L42
'ShopifyException.Errors' is obsolete: 'This property has been moved to the ShopifyHttpException and will be removed in a future version of ShopifySharp.'
Build: ShopifySharp/Infrastructure/ShopifyException.cs#L43
'ShopifyException.RawBody' is obsolete: 'This property has been moved to the ShopifyHttpException and will be removed in a future version of ShopifySharp.'
Build: ShopifySharp/Infrastructure/ShopifyException.cs#L44
'ShopifyException.RequestId' is obsolete: 'This property has been moved to the ShopifyHttpException and will be removed in a future version of ShopifySharp.'
Unit tests (.NET Framework)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Artifacts

Produced during runtime
Name Size
artifacts Expired
5.67 MB
framework-test-results Expired
150 KB