-
-
Notifications
You must be signed in to change notification settings - Fork 416
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
chore: remove unused cookies code #1508
Conversation
throw new errors.UnknownError(`Error parsing cookies from result: '${jsCookies}'`); | ||
} | ||
}; | ||
|
||
commands.setCookie = async function setCookie (cookie) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it also makes sense to move this helper to cookies.js module
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it would also be interesting to know if the remote debugger has some helpers to set cookies
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea, probably can. I found https://github.com/WebKit/WebKit/blob/main/Source/JavaScriptCore/inspector/protocol/Page.json#L191-L196, so am thinking to play something later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as another pr
cc @boneskull as cookies stuff's update |
## [4.18.2](v4.18.1...v4.18.2) (2023-02-26) ### Miscellaneous Chores * remove unused cookies code ([#1508](#1508)) ([1f5e2c4](1f5e2c4))
🎉 This PR is included in version 4.18.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
In #1484 (comment), I noticed https://github.com/appium/appium-xcuitest-driver/blob/master/lib/commands/cookies.js had
getCookies
,deleteCookie
and_deleteCookie
via remote-debugger's WebInspector code instead of JS script right now.So it probably would be nice to remove the unused methods in
web.js
.I checked in a real device, so not in a simulator yet. I'll do a double-check with a simulator later if the current mater uses which code.