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.

Sign up for GitHub

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

Jump to bottom

enable security.txt route also for no-ds setup #7191

Merged
merged 1 commit into from
Jul 7, 2023
Merged

enable security.txt route also for no-ds setup #7191

merged 1 commit into from
Jul 7, 2023

Conversation

fm3
Copy link
Member

@fm3 fm3 commented Jul 7, 2023

follow-up for #7182

in some instances, the routes file is changed to one of these (to disable the datastore and/or tracingstore routes if those modules are disabled)

@fm3 fm3 added the backend label Jul 7, 2023
@fm3 fm3 requested a review from frcroth July 7, 2023 12:37
@fm3 fm3 self-assigned this Jul 7, 2023
frcroth
frcroth approved these changes Jul 7, 2023
View reviewed changes
Copy link
Member

@frcroth frcroth left a comment

Choose a reason for hiding this comment

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

Nice! I was wondering why it didn't work on wkorg

@fm3 fm3 merged commit 1af77c3 into master Jul 7, 2023
@fm3 fm3 deleted the security-txt-no-ds branch July 7, 2023 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@frcroth frcroth frcroth approved these changes

Assignees

@fm3 fm3

Labels
backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fm3 @frcroth