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

Cookies Request #158

Closed
wants to merge 5 commits into from
Closed

Cookies Request #158

wants to merge 5 commits into from

Conversation

danielnehrig
Copy link

@danielnehrig danielnehrig commented Jul 13, 2022

TODO fix tests

danielnehrig and others added 4 commits July 13, 2022 20:27
Signed-off-by: Daniel Nehrig <daniel.nehrig@dnehrig.com>
Signed-off-by: Daniel Nehrig <daniel.nehrig@dnehrig.com>
Signed-off-by: Daniel Nehrig <daniel.nehrig@dnehrig.com>
@danielnehrig
Copy link
Author

i've been a bit busy i'll come back to this soon

@danielnehrig danielnehrig mentioned this pull request Oct 7, 2022
@danielnehrig
Copy link
Author

danielnehrig commented Oct 7, 2022

i'm back i'll pick this up in the upcoming week
edit: ok soon is postponed lol

@arctic-hen7
Copy link
Member

@danielnehrig how is this going? (Btw, I noticed while looking through your code here that the cookie setting system won't work, I don't think, because the request provided to get_request_state etc. is just a clone, you'll need to use the header setting API for that I reckon.)

@danielnehrig
Copy link
Author

@danielnehrig how is this going? (Btw, I noticed while looking through your code here that the cookie setting system won't work, I don't think, because the request provided to get_request_state etc. is just a clone, you'll need to use the header setting API for that I reckon.)

thanks for that hint i wasn't able to test the implementation at that time due to conflicts with my projects cargo release version and the perseus upstream version

there should be more then enough time for me to pick this up again after the holidays probably start next year!

@arctic-hen7
Copy link
Member

Sounds great! No particular rush. I don't think it will impact this PR too much, but be aware that the capsules PR rewrites over 10k lines of the Perseus core, basically constituting a full rewrite. There will probably be a few merge conflicts! (Nothing about Request has changed though.)

@danielnehrig danielnehrig mentioned this pull request Jan 2, 2023
@arctic-hen7
Copy link
Member

@danielnehrig is it likely you'll continue work on this any time soon, or shall I close it for now?

@danielnehrig
Copy link
Author

i'm so sorry i'm deeply burried in work at the moment
i'll revisit stuff soon if i find time :/ for now i close this since it needs to be properly done anyways

@arctic-hen7
Copy link
Member

No problem, let me know if you can in future, but otherwise no worries, I completely understand.

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.

None yet

2 participants