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

razor headers restored in reverse order #5

Closed
jerome-raffalli opened this issue May 25, 2021 · 3 comments
Closed

razor headers restored in reverse order #5

jerome-raffalli opened this issue May 25, 2021 · 3 comments
Assignees
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@jerome-raffalli
Copy link

Hello,
I think there is a problem in CSHtmlMinifier.cs, because headers are restored in reverse order.
Example:

@page
@inherits "Web_View"
...

is restored as

@inherits "Web_View"
@page
...

have a nice day,
Jerome

@jerome-raffalli
Copy link
Author

Proposed patch on line 60 replace :
headers.Add(content);
with
headers.Insert(0, content);

@TwentyFourMinutes
Copy link
Owner

Looks like I somehow never noticed that, I'll fix it ASAP. You can also create a Pull Request if you think that this already resolves the issue.

@TwentyFourMinutes TwentyFourMinutes added bug Something isn't working good first issue Good for newcomers labels May 27, 2021
@TwentyFourMinutes TwentyFourMinutes self-assigned this May 27, 2021
@TwentyFourMinutes
Copy link
Owner

Your issue is fixed in version 2.2.2, thanks for reporting it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants