Build log url from the project organization id - #6
Merged
Merged
Conversation
The log url was derived from an organization slug parsed out of project.subscription.subscription_management_uri. Upsun returns that field as an empty string for every project checked, so deployment statuses never got a log_url and each run logged "Unable to determine Upsun owner organization from subscription_management_uri". Use project.organization instead. The console accepts the organization id in place of the slug, as its own project links already do. Assisted-by: Claude <noreply@anthropic.com>
kasperg
force-pushed
the
fix/log-url-organization-id
branch
from
September 30, 2026 07:35
a50cc97 to
78e332f
Compare
kasperg
marked this pull request as ready for review
September 30, 2026 07:44
|
🎉 This PR is included in version 0.1.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Builds the deployment status log url from
project.organizationinstead of an organization slug parsed out ofproject.subscription.subscription_management_uri.That field is an empty string for all 20 Reload projects the Upsun API returns, and the live script run on an sample project sees the same. So no deployment status got a
log_url, and every run logged "Unable to determine Upsun owner organization from subscription_management_uri".project.organizationis set on every project and is in Upsun's activity script reference example.The console accepts the organization id where the slug went: the API's own
#uiproject link uses the id, and the log url with the id opens the same log as with the slug url. Upsun's API spec marksorganizationas required but nullable; when it is null, the status is sent withoutlog_url, as before.Verified locally:
npm run check. New tests cover the log url and its absence without an organization; nothing testedlog_urlbefore.