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

chore: remove "/wd/hub" prefix to improve compatibility #526

Merged
merged 2 commits into from
Aug 18, 2023

Conversation

howard1005
Copy link
Contributor

This PR aims to enhance compatibility with previous appium versions by removing the "/wd/hub" prefix from URIs.

I am currently using Appium version 1.22.3, and I faced compatibility issues with the latest version of UIAutomator2 regarding the URI. I discovered that requests with the prefix "/wd/hub" were not compatible. By implementing a change to remove this specific prefix from incoming requests, I was able to enhance compatibility, allowing the latest version of UIAutomator2 to work with previous versions of Appium as well. This solution ensures a more seamless integration between these different versions, bridging the gap that existed due to the URI inconsistency.

@howard1005
Copy link
Contributor Author

@mykola-mokhnach , I've addressed the comments and made the necessary changes. Thank you!

@mykola-mokhnach mykola-mokhnach changed the title chore : remove "/wd/hub" prefix to improve compatibility chore: remove "/wd/hub" prefix to improve compatibility Aug 18, 2023
@mykola-mokhnach mykola-mokhnach merged commit ac450c3 into appium:master Aug 18, 2023
10 of 11 checks passed
github-actions bot pushed a commit that referenced this pull request Aug 18, 2023
## [5.12.3](v5.12.2...v5.12.3) (2023-08-18)

### Miscellaneous Chores

* remove "/wd/hub" prefix to improve compatibility ([#526](#526)) ([ac450c3](ac450c3))
@github-actions
Copy link

🎉 This PR is included in version 5.12.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants