Repository navigation
fix: Render the report, skip Python checks cleanly and keep the API key private (0.8.8) - #75
Merged
Merged
Conversation
…ntent list summarise_users() stored the full content listing, and get_quarto_locked_user_apps() ran if (is.na()) on it, so report.qmd failed to render on every licensed server. It now stores the locked-user counts. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BEZ4ZUv51gFQjAe1WePbJF
…ct's config deploy_python() passed --api-key to rsconnect, where any user on the machine could read it with ps. It now goes through CONNECT_SERVER and CONNECT_API_KEY. check() also gives rsconnect a temporary config folder, so the account file holding the key is deleted after the audit. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BEZ4ZUv51gFQjAe1WePbJF
The Flask check fell through to the deploy and the Quarto Python check had no skip, so both were reported as failed; Streamlit and Voila returned before writing a log row and were missing from the results. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BEZ4ZUv51gFQjAe1WePbJF
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
1 participant
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.
0.8.7 → 0.8.8
report.qmdrenders on licensed servers. The summary stores how much content each locked user owns, not the full content list (report.qmd fails to render: get_quarto_locked_user_apps() errors on licensed servers #64).CONNECT_API_KEY, not on the command line, and the rsconnect account that holds it is kept in a temporary folder (The Connect API key shows in the process list and stays in rsconnect's account file #66).Closes #64, closes #65, closes #66
🤖 Generated with Claude Code