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

Add retry for oauth metadata endpoint #4254

Closed
wants to merge 15 commits into from

Conversation

squiishyy
Copy link
Contributor

Tracking issue

Describe your changes

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Screenshots

Note to reviewers

@welcome
Copy link

welcome bot commented Oct 17, 2023

Thank you for opening this pull request! 🙌

These tips will help get your PR across the finish line:

  • Most of the repos have a PR template; if not, fill it out to the best of your knowledge.
  • Sign off your commits (Reference: DCO Guide).

squiishyy and others added 15 commits October 18, 2023 11:16
…ttprequestwithretry, need to work on config mappingn still, tests in

Signed-off-by: squiishyy <[email protected]>
Signed-off-by: Andrew Dye <[email protected]>
Signed-off-by: squiishyy <[email protected]>
Signed-off-by: Andrew Dye <[email protected]>
Signed-off-by: squiishyy <[email protected]>
* Format go imports (flytecopilot)

Signed-off-by: Andrew Dye <[email protected]>

* Fix spelling

Signed-off-by: Andrew Dye <[email protected]>

---------

Signed-off-by: Andrew Dye <[email protected]>
Signed-off-by: squiishyy <[email protected]>
Signed-off-by: Andrew Dye <[email protected]>
Signed-off-by: squiishyy <[email protected]>
---------

Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: squiishyy <[email protected]>
---------

Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: squiishyy <[email protected]>
* Start metrics server in a goroutine

Signed-off-by: Eduardo Apolinario <[email protected]>

* make lint

Signed-off-by: Eduardo Apolinario <[email protected]>

---------

Signed-off-by: Eduardo Apolinario <[email protected]>
Co-authored-by: Eduardo Apolinario <[email protected]>
Signed-off-by: squiishyy <[email protected]>
@squiishyy squiishyy force-pushed the squiishyy/oatuh2-retry branch from 231c04b to ac0ab9a Compare October 18, 2023 18:17
@squiishyy
Copy link
Contributor Author

Borked it with a rebase, cherrypicked to here: #4259

@squiishyy squiishyy closed this Oct 18, 2023
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.

7 participants