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

Add exception routes configuration to reload module #1150

Open
Autre31415 opened this issue Apr 7, 2022 · 0 comments · May be fixed by #1457
Open

Add exception routes configuration to reload module #1150

Autre31415 opened this issue Apr 7, 2022 · 0 comments · May be fixed by #1457
Assignees
@kethinov
Labels
enhancement Planned feature good first issue If you're new to the project, this would be a good issue to try resolving first help wanted

Comments

@Autre31415
Copy link
Member

Similar to #1149, when setting up frameworks that spin up their own dynamic routes we can end up in situations where the reload script tag gets appended to things that it shouldn't. A quick solution for niche scenarios like this is to allow users to configure a route or prefix of routes where the reload script tag won't get appended to a http response object.
@Autre31415 Autre31415 added enhancement Planned feature help wanted good first issue If you're new to the project, this would be a good issue to try resolving first labels Apr 7, 2022
@Autre31415 Autre31415 changed the title Add exception routes configuration for reload module Add exception routes configuration to reload module Apr 7, 2022
@kethinov kethinov assigned kethinov and unassigned Autre31415 Jun 26, 2023
@alallier alallier linked a pull request Aug 28, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kethinov kethinov

Labels
enhancement Planned feature good first issue If you're new to the project, this would be a good issue to try resolving first help wanted
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added the ability to exclude frontend reload script injection via configuration alallier/roosevelt
2 participants
@kethinov @Autre31415