search

appium / appium

Cross-platform automation framework for all kinds of apps, built on top of the W3C WebDriver protocol
http://appium.io/
Apache License 2.0
18.86k stars 6.08k forks source link

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

Open eglitise opened 7 months ago

eglitise commented 7 months ago

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)

KazuCocoa commented 7 months ago

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 commented 7 months ago

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

KazuCocoa commented 7 months ago

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.