-
Notifications
You must be signed in to change notification settings - Fork 126
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
DEVPROD-11847 remove create version route #8368
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we do this, we also have to delete evergreen create-version
since it uses this route. We should also close DEVPROD-11844 and update the email that was just sent out about evergreen create-version
since the functionality will not exist anymore.
Oooooof okay Idk why I assumed that used a different endpoint... let me consider bringing it back and just fixing the route, since we just told Trevor about it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The decision making is chaotic but I have no opposition to deleting the feature 😄
apologies for chaos haha |
DEVPROD-11847
Description
This route isn't authed strictly enough, allows you to bypass adhoc versions, and use admin only variables.
Testing
Wrote a test for admin only variables to make sure we think about our requesters and whether they should have admin-only variable access explicitly.
Documentation
ran swaggo