Skip to content

[5] Fix for IAP Auth Issues#6

Merged
ryanm101 merged 1 commit intomasterfrom
5_IAPFix
Jan 21, 2022
Merged

[5] Fix for IAP Auth Issues#6
ryanm101 merged 1 commit intomasterfrom
5_IAPFix

Conversation

@ryanm101
Copy link
Copy Markdown
Contributor

@ryanm101 ryanm101 commented Jan 20, 2022

Fix for #5

@ryanm101
Copy link
Copy Markdown
Contributor Author

@solsson Do you want to try this and see if it solves your issue?

@solsson
Copy link
Copy Markdown
Contributor

solsson commented Jan 20, 2022

Confirmed! Well done 👏

It's interesting that the success rate with GCP_IAAP_AUTH_TOKEN seemed to degrade day by day. Today it was really terrible. Is GCP in the midst of a gradual rollout that de-supports cookie auth? Did you get an answer to your ticket?

@ryanm101
Copy link
Copy Markdown
Contributor Author

Apparently they will look into it and update me by the 24th of Jan..

@ryanm101 ryanm101 merged commit d0ddf74 into master Jan 21, 2022
@ryanm101 ryanm101 deleted the 5_IAPFix branch January 21, 2022 09:14
@solsson
Copy link
Copy Markdown
Contributor

solsson commented Jan 21, 2022

Apparently they will look into it and update me by the 24th of Jan..

I think this PR is an improvement anyway, but it's interesting to know if there are changes to IAP in general. There's no hints of that in https://cloud.google.com/iap/docs/release-notes.

@ryanm101
Copy link
Copy Markdown
Contributor Author

ryanm101 commented Jan 21, 2022

| There's no hints of that in https://cloud.google.com/iap/docs/release-notes.

Yes I KNOW!!!!

It was very annoying trying to figure out where the problem lay as we'd recently made some infra changes on our side. You're reporting of the issue here helped us confirm it wasn't something we'd done, so thank you for that.

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