Skip to content

Conversation

@dosomder
Copy link
Contributor

Otherwise, nipap-www will always automatically create the user test with password test.

Fix for #572

Copy link
Member

Choose a reason for hiding this comment

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

Is there a point in keeping the "@Local" part of the username? Shouldn't that be removed as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is it possible / is there an use-case, to use it without local auth (i.e. with ldap auth)? If yes, then it should be removed. If however nipap-www is intended to always connect using local auth, it should be kept in my opinion.

Copy link
Member

Choose a reason for hiding this comment

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

Right, but the sed line we have right now in postinst already adds "@Local", so it won't really make a difference to the end result. What it does though is look confusing, IMHO :P
We are only talking about the default value here, anyone is free to add @Local or something else to their configuration.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've removed it. Do you want to remove everything up to the ip, since sed adds that? I.e.
xmlrpc_uri = 127.0.0.1:1337

@dosomder dosomder force-pushed the 572-default-www-user branch from d801164 to e333a66 Compare August 21, 2014 07:22
Otherwise, nipap-www will always automatically create the user test with
password test
@dosomder dosomder force-pushed the 572-default-www-user branch from e333a66 to c48a4bb Compare August 21, 2014 12:17
plajjan added a commit that referenced this pull request Aug 28, 2014
@plajjan plajjan merged commit 6bab59f into SpriteLink:master Aug 28, 2014
@plajjan plajjan added this to the Version 0.27 - Icarus milestone Aug 28, 2014
@plajjan plajjan self-assigned this Aug 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants