Skip to content

OpenDJ doc specify "localhost" as default#127

Merged
vharseko merged 1 commit into
OpenIdentityPlatform:masterfrom
Marsonge:master
Jun 10, 2020
Merged

OpenDJ doc specify "localhost" as default#127
vharseko merged 1 commit into
OpenIdentityPlatform:masterfrom
Marsonge:master

Conversation

@Marsonge

Copy link
Copy Markdown
Member

The current code can return an IP or hostname instead of localhost, which means that using a script in local on a server which only listens to 127.0.0.1 can fail without the -h flag.

Forgerock's OpenDJ is pretty clear about it: the default value is "localhost.localdomain". However, using the full "localhost.localdomain" string seems to not work for at least some of the scripts, if not all of them; thus, I highly recommend using solely "localhost".

This solves multiple problems:

  • Retrocompatibility issues with servers migrating to OpenIdentity's OpenDJ that bind their server to localhost but do not use the hostname flag in their scripts
  • Avoids a discrepancy between OpenIdentity's OpenDJ and Forgerock's documentation
  • Avoids unpredictable behaviour when there are multiple network cards on a machine: the former code could pull any IP address from there, meaning the IP address used couldn't be reliably known.

The current code can return an IP or hostname instead of localhost, which means that using a script in local on a server which only listens to 127.0.0.1 can fail without the -h flag.
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