NXDRIVE-2901: Authorization Error for OAuth

[gitofanindya]

No description provided.

[sourcery-ai]

PR Type: Enhancement

PR Summary: The pull request aims to address an authorization error in the OAuth flow within the NXDrive application. It modifies the OAuth2 initialization process by ensuring the 'verify' parameter is correctly set in the 'subclient_kwargs'. This change is intended to enhance the robustness and reliability of the OAuth authorization process.

Decision: Comment

📝 Type: 'Enhancement' - not supported yet.

• Sourcery currently only approves 'Typo fix' PRs.

✅ Issue addressed: this change correctly addresses the issue or implements the desired feature.

No details provided.

✅ Small diff: the diff is small enough to approve with confidence.

No details provided.

General suggestions:

• Ensure that the modification does not introduce any side effects in the OAuth flow, especially in environments with different security requirements.
• Review the naming and initialization pattern of 'subclient_kwargs' to ensure it aligns with the project's coding standards and practices.
• Consider the impact of this change on existing deployments and whether any additional configuration or documentation updates are needed to support the change.

Thanks for using Sourcery. We offer it for free for open source projects and would be very grateful if you could help us grow. If you like it, would you consider sharing Sourcery on your favourite social media? ✨

Share Sourcery

• X
• Mastodon
• LinkedIn
• Facebook

---

Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.25%. Comparing base (81d7886) to head (fb14740).
Report is 6 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4616      +/-   ##
==========================================
+ Coverage   49.17%   49.25%   +0.07%     
==========================================
  Files          94       94              
  Lines       15681    15699      +18     
==========================================
+ Hits         7711     7732      +21     
+ Misses       7970     7967       -3     

Flag	Coverage Δ
functional	38.08% <100.00%> (+0.07%)	⬆️
integration	2.06% <0.00%> (-0.01%)	⬇️
unit	37.57% <17.39%> (-0.05%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mr-shekhar]

👍