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

Update deprecated calls to HostAndPort.getHostText() #389

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shs96c
Copy link
Contributor

@shs96c shs96c commented Aug 18, 2017

The getHostText method is deprecated, and scheduled
for deletion. Although LittleProxy continues to
support Java 7, using this deprecated method causes
it to fail when used to Guava 23 (the latest release)

Fortunately, getHost returns exactly the same value
and is present in the latest releases.

The `getHostText` method is deprecated, and scheduled
for deletion. Although LittleProxy continues to
support Java 7, using this deprecated method causes
it to fail when used to Guava 23 (the latest release)

Fortunately, `getHost` returns exactly the same value
and is present in the latest releases.
@shs96c
Copy link
Contributor Author

shs96c commented Aug 18, 2017

Java 7 has been dead for a long time, and Java 9 is coming, but I'm offering this as an alternative to #388

Copy link

@xtj7 xtj7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as in PR 388, we really need it to work in projects with more recent Guava versions.

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

Successfully merging this pull request may close these issues.

2 participants