Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix WIF api call options #174

Merged
merged 6 commits into from
Nov 25, 2024
Merged

Fix WIF api call options #174

merged 6 commits into from
Nov 25, 2024

Conversation

asatwal
Copy link
Collaborator

@asatwal asatwal commented Nov 19, 2024

Trello Ticket

The initialisation of options to Google API WIF calls is causing errors on GIT.

Currently this line:

Google::Apis::RequestOptions.new(retries: 3)

Causes the following errors in the options:

#<struct Google::Apis::RequestOptions
 authorization={retries=>3},
 retries=nil
...

This error has been fixed in the PR and the authorization also setup properly

@asatwal asatwal self-assigned this Nov 19, 2024
@asatwal asatwal force-pushed the fix-wif-api-call-options branch from 2b683a1 to 848bed7 Compare November 22, 2024 15:56
@asatwal asatwal marked this pull request as ready for review November 25, 2024 09:31
@asatwal asatwal requested a review from goodviber November 25, 2024 09:31
Copy link
Contributor

@goodviber goodviber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - perhaps we could remove some of the comments

@asatwal asatwal merged commit 52c1199 into main Nov 25, 2024
5 checks passed
@asatwal asatwal deleted the fix-wif-api-call-options branch November 25, 2024 11:58
asatwal added a commit that referenced this pull request Nov 25, 2024
* Fix retry options to Google API calls that are causing errors in GIT

* Cater for the max elapsed retry durection for all retries in the token expiry leeway

* Fix syntax used for enabling/disabling cop Style/ClassVars

* Fix out of date Gemfile.lock

* Hold durations as integers in constants to avoid use of ActiveSupport

* Fix events_client test stub for WIF
asatwal added a commit that referenced this pull request Nov 26, 2024
* Fix retry options to Google API calls that are causing errors in GIT

* Cater for the max elapsed retry durection for all retries in the token expiry leeway

* Fix syntax used for enabling/disabling cop Style/ClassVars

* Fix out of date Gemfile.lock

* Hold durations as integers in constants to avoid use of ActiveSupport

* Fix events_client test stub for WIF
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants