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

fix: Cookie Handling and Payload Processing #83

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

SachaMorard
Copy link
Contributor

Checklist

  • I have read the Contributor Guide
  • I have read and agree to the Code of Conduct
  • I have added a description of my changes and why I'd like them included in the section below

Description of Changes

Data collection components always received session_start instructions... There was an error redefining the user cookie at each request.

This pull request introduces several improvements and fixes to the edgee_cookies.rs, compute.rs, controller.rs, and proxy.rs files. The changes aim to enhance the handling of cookies, improve payload processing, and ensure better compliance with the application's requirements.

Changes:

  1. edgee_cookies.rs:

    • The methods of this script are now requiring a response_parts parameter instead of response_headers
  2. compute.rs:

    • edgee_cookie::get can now use the response_parts parameter instead of response_headers, and it fixes the bug we had on the session cookie set.
  3. controller.rs:

    • Improved the edgee_client_event function to handle incoming client events more effectively and use response_parts object
  4. proxy.rs:

    • Factorizing following changes to the other parts

Related Issues

no issue

@SachaMorard SachaMorard merged commit 8f0ba0e into main Oct 1, 2024
2 checks passed
@SachaMorard SachaMorard deleted the fix/session-update-in-cookie branch October 1, 2024 03:59
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.

1 participant