Skip to content

Port is not set using com.aftership.tracking.http.Request.setDomain() #17

@CharlyRien

Description

@CharlyRien

Hello,
since the 10.0.0 version we have found an issue with the way request are handled, that make us impossible to add a port in the request.
Therefore when trying to test for example with a mocked Aftership server with a specific port, that's just not working anymore.

The "problem" seems to come from this method

    public void setDomain(final String domain) throws Exception {
        URIBuilder validateURI = new URIBuilder(domain);
        uriBuilder.setHost(validateURI.getHost());
        uriBuilder.setScheme(validateURI.getScheme());
    } 

in class com.aftership.tracking.http.Request

that does not take into account the port, as before.
I don't have a workaround right now (except rewriting everything) so i was wondering why this change appeared, and if that was possible to put back the previous behavior?

Thanks in advance :)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions