"Comma separated list" is incorrect

[p0deje]

"Comma separated list" is incorrect

[mocleiri]

Thanks for your feed back.

I think the comma space thing is just a rendering issue (i.e. I thought it looked nicer to have the space between the user names).

The ACL itself does a split on just the comma: https://github.com/jenkinsci/github-oauth-plugin/blob/master/src/main/java/org/jenkinsci/plugins/GithubRequireOrganizationMembershipACL.java#L198

look at line 201 and you will see I trim the parts.

[p0deje]

Oh, now I understand.

Still, when I enter p0deje, some_other_user to Admin User Names, I have admin rights.
But when I enter p0deje,some_other_user to Admin User Names I loose admin rights and have only READ permissions (just because I enabled Grant READ permissions to all Authenticated Users

Do you have any idea why could this happen?

[mocleiri]

I think this issue was fixed in the 0.9 plugin release. What version of the plugin are you using?

[p0deje]

I use 0.9

[mocleiri]

I'm no longer maintaining the plugin.

@skottler is but using the main repository here: https://github.com/jenkinsci/github-oauth-plugin

Normally jenkins projects use jira for the issues but I never turned them off here. But I think its confusing for reporters to report here.

I want to turn off the issues on this repository so can you refile the ticket into the jenkins jira for the github-oauth component?

Unfortunately there doesn't seem to be a way to make the existing issues read-only so I can disable issues without all of the existing reported and resolved issues vanishing.

Thanks