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

Regex validation in GitHub node prevents usage of enterprise GitHub #13491

Open
emathias opened this issue Feb 25, 2025 · 1 comment
Open

Regex validation in GitHub node prevents usage of enterprise GitHub #13491

emathias opened this issue Feb 25, 2025 · 1 comment
Labels
in linear Issue or PR has been created in Linear for internal review In Review We are reviewing the issue

Comments

@emathias
Copy link

Hi,

I'd like to report a small issue regarding GitHub integration. I'm running n8n Version 1.79.3

At the moment, the regex validations in lines:

regex: 'https:\\/\\/github.com\\/([-_0-9a-zA-Z]+)(?:.*)',

and

regex: 'https:\\/\\/github.com\\/(?:[-_0-9a-zA-Z]+)\\/([-_.0-9a-zA-Z]+)(?:.*)',

Prevent the usage of the GH Enterprise server.

The fix:
IMHO, 'github.com' should not be hardcoded in regex. Instead, it can be extracted from credential. An extra field in the UI which configures the node would also do, but since GH credentials are instance bound, getting it from credential config would make more sense.

Thanks and kind regards

ps: getting owner and repo by name seems to work

@Joffcom
Copy link
Member

Joffcom commented Feb 25, 2025

Hey @emathias,

We have created an internal ticket to look into this which we will be tracking as "GHC-953"

@Joffcom Joffcom added in linear Issue or PR has been created in Linear for internal review In Review We are reviewing the issue labels Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in linear Issue or PR has been created in Linear for internal review In Review We are reviewing the issue
Projects
None yet
Development

No branches or pull requests

2 participants