Skip to content

Add timeout#15

Open
braedongough wants to merge 4 commits intopeter-murray:mainfrom
braedongough:feat/make-timeout-configurable
Open

Add timeout#15
braedongough wants to merge 4 commits intopeter-murray:mainfrom
braedongough:feat/make-timeout-configurable

Conversation

@braedongough
Copy link
Copy Markdown

Passes an optional timeout for configuring token expiry.

@braedongough
Copy link
Copy Markdown
Author

hi @peter-murray! Is this PR suitable for review? 😄

@peter-murray
Copy link
Copy Markdown
Owner

I do not think so, as the last time I looked to this by the time I got to the bottom, this timeout is not the timeout you think it is...

I need to re-review the underlying functionality and test it to confirm before exposing this.

@braedongough
Copy link
Copy Markdown
Author

I thought I double-checked this but happy to have you confirm when you can! 😄

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