Skip to content

fix: update error message to handle enterprise servers - #863

Merged
woodruffw merged 2 commits into
zizmorcore:mainfrom
tiymat:fix-error-message
May 28, 2025
Merged

fix: update error message to handle enterprise servers#863
woodruffw merged 2 commits into
zizmorcore:mainfrom
tiymat:fix-error-message

Conversation

@tiymat

@tiymat tiymat commented May 27, 2025

Copy link
Copy Markdown
Contributor

While using zizmor at work today I noticed that when specifying --gh-hostname and failing to collect inputs from an on-prem GitHub Enterprise Server, zizmor would emit a message making it seem like it was trying to connect to github.com.

@woodruffw woodruffw left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice, thanks a ton @tiymat!

Signed-off-by: William Woodruff <william@yossarian.net>
@woodruffw
woodruffw enabled auto-merge (squash) May 28, 2025 02:50
@woodruffw woodruffw added cli bugfix Fixes a known bug labels May 28, 2025
@woodruffw
woodruffw merged commit 3545ae6 into zizmorcore:main May 28, 2025
@funnelfiasco funnelfiasco added the ghes GitHub Enterprise Server issues label May 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes a known bug cli ghes GitHub Enterprise Server issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants