Skip to content
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

Create support for username-password passing without tag value limitations #291

Open
walsw opened this issue Aug 15, 2023 · 3 comments
Open
Labels
enhancement New feature or request

Comments

@walsw
Copy link

walsw commented Aug 15, 2023

What feature do you want to see added?

Create support for username-password passing without tag value limitations.
Now the DOMAIN\username syntax, common for NetBIOS notation is impossible to store in secret manager as the username is stored as a tag value pair, preventing the use of backslash (''). If username was stored as secrets value, that would not be a limitation.
We currently cannot use the plugin to store our Checkmarx username-password as we mandate it is AD authenticated (We now have to create the Jenkins native username/password to allow our username to be prefixed with the domainname\ )

Upstream changes

No response

@walsw walsw added the enhancement New feature or request label Aug 15, 2023
@chriskilding
Copy link
Contributor

Depending on how your AD auth is set up, you may be able to use the new username@domain syntax for the username instead of the DOMAIN\username syntax.

I know this works for general sign-in if your organization uses Azure AD, so I think it should work for what you are trying to do with Checkmarx as well. (I don't know how traditional on-premise AD would behave if given a new-style username.)

Give this a go and let me know if it works

@walsw
Copy link
Author

walsw commented Aug 17, 2023

Had already tried the @Domain notation. That is allowed in tag value, but Checkmarx doesn't understand it. It probably verifies it against local database as an username looking like email address. In the windows world they often get treated as equivalents, but they are not. The DOMAIN\username is an (older) notation based of NetBios (which can be disabled). Where the username@fqdn domain notation is a UPN notation. Possible to map a user against multiple domains. regardless of this, it would be better to keep the username as a secret from a security perspective. Tags are unencrypted and less protected from prying eyes than secrets. Knowing a valid username is half the battle when you try to brute force access by guessing username-password combinations.

@chriskilding
Copy link
Contributor

It sounds like the first port of call is to raise this with Checkmarx, to get a definitive answer on how Checkmarx handles user@domain usernames. They may well decide that there is a bug that needs fixing. Could you provide more details about which specific Checkmarx products or components you are using with your Jenkins job? (That will indicate whether we need to raise a GitHub Issue on the relevant repo, or send an email to Checkmarx support.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants