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

Endpoints are left in insecure states during some updates #219

Closed
euank opened this issue May 8, 2023 · 1 comment
Closed

Endpoints are left in insecure states during some updates #219

euank opened this issue May 8, 2023 · 1 comment
Labels
area/controller Issues dealing with the controller bug Something isn't working needs-triage Issues that need triage

Comments

@euank
Copy link
Contributor

euank commented May 8, 2023

What happened

This issue is similar to #208, and mentioned there, but seems worth filling separately for clarity.

Certain updates will leave the edge in an insecure state, even though a security module is configured for the entire duration.

For example, after creating an ngrok edge, a change that tries to replace OIDC with OAUTH will apply without any changes to the serving status of the backend. However, while the change is being applied, the edge will have neither OIDC nor OAUTH configured, and thus may route unauthorized requests.

This is made worse by the fact that changing from OIDC to OAUTH will actually error out terminally, and thus the edge will just be left insecure forever in the above scenario (until a human operator deletes it, or revers the change, or takes some other action).

What you think should happen instead

When an update occurs, the update should be atomic such that either the old or new config applies (i.e. when changing from OAuth to SAML, every request should observe one or the other, not neither or both at once).

@euank euank added bug Something isn't working area/controller Issues dealing with the controller needs-triage Issues that need triage labels May 8, 2023
@jrobsonchase jrobsonchase self-assigned this May 16, 2023
@jrobsonchase jrobsonchase removed their assignment Jun 6, 2023
@euank euank changed the title Endpoints are left in insecure states during creation and some updates Endpoints are left in insecure states during soem updates Jun 26, 2023
@euank euank changed the title Endpoints are left in insecure states during soem updates Endpoints are left in insecure states during some updates Jun 26, 2023
@alex-bezek
Copy link
Collaborator

This was completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/controller Issues dealing with the controller bug Something isn't working needs-triage Issues that need triage
Projects
None yet
Development

No branches or pull requests

3 participants