Skip to content

Conversation

@tqq1994516
Copy link

@tqq1994516 tqq1994516 commented Oct 16, 2025

@gbj
Copy link
Collaborator

gbj commented Oct 20, 2025

Did you test this? When I try it out, your modified example panics on all static file requests.

@tqq1994516
Copy link
Author

Did you test this? When I try it out, your modified example panics on all static file requests.

I verified it at the time. I don't know if it was a cache issue or something else, but now it does panic. I will continue to fix it.

@gbj
Copy link
Collaborator

gbj commented Oct 24, 2025

It still wasn't setting the headers as requested, because there was actual no ownership system/context set up -- easy enough fix so I thought I'd just modify it myself rather than bugging you!

Thanks for the PR.

@tqq1994516
Copy link
Author

tqq1994516 commented Oct 25, 2025

It still wasn't setting the headers as requested, because there was actual no ownership system/context set up -- easy enough fix so I thought I'd just modify it myself rather than bugging you!

Thanks for the PR.

That shouldn't be the case. When I ran that example according to my PR, I saw the headers I wanted.
image
image
image
image
Could you tell me the role of the newly added Owner? I think this can help me understand leptos better.

@tqq1994516
Copy link
Author

tqq1994516 commented Oct 25, 2025

Oh, I found that the addition of this header in my PR is unstable; every time I refresh, it may not be added as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants