-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fix webapp startup if no docker IP's set. #8655
Conversation
@Nullable @Named("che.docker.ip") String internalIpOfContainers, | ||
@Named("che.port") int chePort, | ||
@Nullable @Named("che.docker.ip.external") String externalIpOfContainers, | ||
@Nullable @Named("che.singleport.wildcard_domain.host") String wildcardHost) { | ||
if (internalIpOfContainers == null && externalIpOfContainers == null) { | ||
if (isSinglePortEnabled && internalIpOfContainers == null && externalIpOfContainers == null) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we move this check into SinglePortHostnameBuilder
?
@Nullable @Named("che.docker.ip") String internalIpOfContainers, | ||
@Nullable @Named("che.docker.ip.external") String externalIpOfContainers, | ||
@Nullable @Named("che.docker.network") String dockerNetwork, | ||
@Nullable @Named("che.singleport.wildcard_domain.host") String wildcardHost) { | ||
if (internalIpOfContainers == null && externalIpOfContainers == null) { | ||
if (isSinglePortEnabled && internalIpOfContainers == null && externalIpOfContainers == null) { | ||
throw new IllegalStateException( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think ConfigurationException
would be better here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree
What does this PR do?
Fix webapp startup if no docker_ip or external_docker_ip is set. That can be possible when running without CLI.
What issues does this PR fix or reference?
N//A
Release Notes
N/A
Docs PR
N/A