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

feature: add flag for cloud providers to say whether they support the /sessions endpoint, to avoid making calls that we know won't work #201

Open
1 task
KritikaSahare1611 opened this issue Nov 15, 2021 · 2 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@KritikaSahare1611
Copy link

KritikaSahare1611 commented Nov 15, 2021

Appium or Appium Desktop?

You are reporting an issue at the Appium Desktop repository. Appium Desktop is a wrapper around Appium. If you are having trouble running tests, it is much more likely that the problem you are encountering is not a problem with Appium Desktop but with Appium. For that reason we require you to have tried your testcase with Appium CLI before reporting issues. Check the checkbox below to confirm that you have proven your issue does not reproduce on Appium itself:

  • I have verified that my issue does not occur with Appium and should be investigated as an Appium Desktop issue

The problem

Not able to attach to the session as Attach to session dropdown is showing blank.

Environment

  • I am running Appium Desktop version <1.22.0>.
  • I am on (Windows):

image

image

@KritikaSahare1611 KritikaSahare1611 changed the title Session id not showing in the inspector for cloud povider browserstack Session id not showing in the inspector for cloud provider browserstack Nov 15, 2021
@KazuCocoa
Copy link
Member

Currently, this behavior is expected since the current implementation depends on the response of /sessions API.
It needs to implement copy&paste session-id and attach to it.

@KazuCocoa KazuCocoa added enhancement New feature or request help wanted Extra attention is needed labels Nov 15, 2021
@jlipps
Copy link
Member

jlipps commented Nov 16, 2021

@KazuCocoa maybe we should have an internal config option for servers, such that if we know a server doesn't support it, we show a "Not supported" card instead of showing the UI. Then if a server eventually supports /session they can open a PR to update their config.

@jlipps jlipps changed the title Session id not showing in the inspector for cloud provider browserstack feature: add flag for cloud providers to say whether they support the /sessions endpoint, to avoid making calls that we know won't work May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants