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

[Feat]: add new endpoint to replace GET /session/:sessionId #19858

Open
eglitise opened this issue Mar 4, 2024 · 3 comments
Open

[Feat]: add new endpoint to replace GET /session/:sessionId #19858

eglitise opened this issue Mar 4, 2024 · 3 comments
Labels

Comments

@eglitise
Copy link
Collaborator

eglitise commented Mar 4, 2024

Based on #19851. These are just ideas and the exact endpoint values are open to discussion.

GET /session/:sessionId => GET /session/:sessionId/appium/info
GET /session/:sessionId/log => GET /session/:sessionId/appium/log (already exists in Selenium as /se/log)
GET /session/:sessionId/log/types => GET /session/:sessionId/appium/appium/types (already exists in Selenium as /se/log/types)

@eglitise eglitise added the Enhancement feature label Mar 4, 2024
@KazuCocoa
Copy link
Member

KazuCocoa commented Mar 4, 2024

for logs, we already re-use /se endpoints to keep client's selenium client's compatibility so the appium one is not necessary for logs

@eglitise
Copy link
Collaborator Author

eglitise commented Mar 4, 2024

Good point, I was not aware of that. Then I guess there's only the session info endpoint left?

@KazuCocoa
Copy link
Member

Yes, i think so.

Btw, I saw the /session/<session id> endpoint was proposed once in the past in the w3c in https://github.com/w3c/webdriver
I don't recall right now about the details but the propose was closed after some discussion.

@eglitise eglitise changed the title [Feat]: add 3 new endpoints to replace non-W3C endpoints [Feat]: add new endpoint to replace GET /session/:sessionId Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants