Skip to content

Check if the cancel function was set before using it#893

Merged
mlsmaycon merged 1 commit into
mainfrom
fix/avoid-panic-when-cancel-is-not-set
May 23, 2023
Merged

Check if the cancel function was set before using it#893
mlsmaycon merged 1 commit into
mainfrom
fix/avoid-panic-when-cancel-is-not-set

Conversation

@mlsmaycon
Copy link
Copy Markdown
Collaborator

Describe your changes

in some cases an IDP device flow expiration time might be shorter than 90s we should check if the cancel context was set before using it

We will need a follow-up to identify and document the IDP with lower defaults.

Issue ticket number and link

fixes #890

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

in some cases an IDP device flow expiration time might be shorter than 90s
we should check if the cancel context was set before using it

We will need a follow-up to identify and document the IDP with lower defaults.

fixes #890
@mlsmaycon mlsmaycon requested a review from gigovich May 23, 2023 13:25
Copy link
Copy Markdown
Contributor

@gigovich gigovich left a comment

Choose a reason for hiding this comment

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

LGTM

@mlsmaycon mlsmaycon merged commit a91297d into main May 23, 2023
@mlsmaycon mlsmaycon deleted the fix/avoid-panic-when-cancel-is-not-set branch May 23, 2023 15:54
pulsastrix pushed a commit to pulsastrix/netbird that referenced this pull request Dec 24, 2023
in some cases an IDP device flow expiration time might be shorter than 90s
we should check if the cancel context was set before using it

We will need a follow-up to identify and document the IDP with lower defaults.

fixes netbirdio#890
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.

windows client daemon panic for pure SSO setup

2 participants