UriComponentsBuilder http/s default port [SPR-15416]

[spring-projects-issues]

Rico Pahlisch opened SPR-15416 and commented

UriComponentsBuilder returns -1 for http and https urls if there is no port given.

Please change the behaviors to return the default ports for both schemas.

@RunWith(SpringRunner.class)
public class UriComponentsTest {

    @Test
    public void test () throws URISyntaxException {
        UriComponents build = UriComponentsBuilder.fromUri(new URI("http://localhost/")).build();
        assertThat(build.getPort()).isEqualTo(80);
    }
}

---

No further details from SPR-15416

[spring-projects-issues]

Juergen Hoeller commented

getPort is currently explicitly defined to return -1 if no specific port has been set. In that sense, it allows for differentiating between an explicit port given versus an applicable default port.

Rossen Stoyanchev, what's your take on this?

[spring-projects-issues]

Rossen Stoyanchev commented

Yes allowing to differentiate between explicit vs no port is useful. It's also not an option to assign defaults. There are issues that can arise from such opinionated defaults, see for example #17368.