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: correctly process NoSuchUserContextException in cookie operations #1940

Merged
merged 2 commits into from
Mar 5, 2024

Conversation

sadym-chromium
Copy link
Collaborator

@sadym-chromium sadym-chromium commented Mar 5, 2024

Parse error when setting/getting cookies to catch NoSuchUserContextException.

  • updated test_cookie_set_partition_user_context to test what it was intended to.

@sadym-chromium sadym-chromium requested a review from OrKoN March 5, 2024 14:15
@sadym-chromium sadym-chromium changed the title feat: correctly process NoSuchUserContextException in cookie operaеions feat: correctly process NoSuchUserContextException in cookie operations Mar 5, 2024
@sadym-chromium sadym-chromium force-pushed the sadym/cookie-unknown-user-context branch from 69e21bc to 08a2caa Compare March 5, 2024 14:54
@OrKoN OrKoN changed the title feat: correctly process NoSuchUserContextException in cookie operations fix: correctly process NoSuchUserContextException in cookie operations Mar 5, 2024
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@sadym-chromium sadym-chromium reopened this Mar 5, 2024
@sadym-chromium sadym-chromium enabled auto-merge (squash) March 5, 2024 16:38
@sadym-chromium sadym-chromium merged commit 7407608 into main Mar 5, 2024
35 checks passed
@sadym-chromium sadym-chromium deleted the sadym/cookie-unknown-user-context branch March 5, 2024 16:42
Lightning00Blade pushed a commit that referenced this pull request Mar 15, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.5.13](chromium-bidi-v0.5.12...chromium-bidi-v0.5.13)
(2024-03-15)


### Features

* `invalid selector` error
([#1985](#1985))
([cba1d35](cba1d35))
* add support for `contexts` in `addInterception`
([#1945](#1945))
([fc76be7](fc76be7))
* start implementing `browsingContext.locateNodes`
([#1970](#1970))
([d61f154](d61f154))
* support `network.continueResponse` authorization
([#1961](#1961))
([528ad63](528ad63))
* support base64 cookie values
([#1933](#1933))
([9d1b975](9d1b975))
* support xpath locators
([#1986](#1986))
([b49184f](b49184f))


### Bug Fixes

* add `authChallenges` to response
([#1919](#1919))
([e4a519a](e4a519a))
* always provide `userContext` in cookie's `partitionKey`
([#1938](#1938))
([0adf6d1](0adf6d1))
* correctly process `NoSuchUserContextException` in cookie operations
([#1940](#1940))
([7407608](7407608))
* don't block on unsubscribed events
([#1954](#1954))
([0abbff8](0abbff8))
* don't throw unhandled errors
([#1996](#1996))
([ab1c6d2](ab1c6d2))
* emit for target id
([#1979](#1979))
([d1091bd](d1091bd))
* the pattern matching logic
([#1995](#1995))
([66010d1](66010d1))
* workaround issue with Script.Target
([#1947](#1947))
([3cc317b](3cc317b))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
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