Skip to content

fix(http): Return error when API key is not set#18

Merged
NautiluX merged 1 commit intomainfrom
fix-no-api-key-set-crash
Sep 10, 2025
Merged

fix(http): Return error when API key is not set#18
NautiluX merged 1 commit intomainfrom
fix-no-api-key-set-crash

Conversation

@NautiluX
Copy link
Member

  • When OMS_PORTAL_API_KEY is unset, the CLI crashed because the error wasn't handled correctly

Card: 869adwyg4

* When OMS_PORTAL_API_KEY is unset, the CLI crashed because the error wasn't handled correctly

Card: [869adwyg4](https://app.clickup.com/t/869adwyg4)
Copy link
Member

@siherrmann siherrmann left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks.

@NautiluX NautiluX merged commit 0ecb39f into main Sep 10, 2025
3 checks passed
@NautiluX NautiluX deleted the fix-no-api-key-set-crash branch September 10, 2025 09:33
siherrmann pushed a commit that referenced this pull request Sep 16, 2025
* When OMS_PORTAL_API_KEY is unset, the CLI crashed because the error wasn't handled correctly

Card: [869adwyg4](https://app.clickup.com/t/869adwyg4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants