fix: use configured RPC proxy credentials and print startup URL - #7
Closed
spacesops wants to merge 14 commits into
Closed
fix: use configured RPC proxy credentials and print startup URL#7spacesops wants to merge 14 commits into
spacesops wants to merge 14 commits into
Conversation
Wire spaced RPC user/password from CLI into AppState so /rpc/spaced no longer uses hardcoded credentials, and add a clear startup console URL that includes the port. Co-authored-by: Cursor <cursoragent@cursor.com>
Author
|
Added handling of options through .env and environment variables in prep for Dockerization. |
… tightening docs/runtime defaults. Store subs-prover calibration in SUBS_DATA_DIR so estimates survive restarts, run calibration in the background when cache is missing, and include a WIF-to-hex utility plus updated publish/run documentation and registry default port alignment. Co-authored-by: Cursor <cursoragent@cursor.com>
Author
|
subs-prover startup CPU cost was heavy and undesired on repeated startups. Create/Load prover calibration file to skip on subsequent starts. |
Surface the same detail in the operator check, space UI tooltips, and local setup hints; ignore subspaces.tar. Co-authored-by: Cursor <cursoragent@cursor.com>
Adapt Zone/fabric APIs and zk-input layout for the new releases, and document the upgrade pins and fabric git pin. Co-authored-by: Cursor <cursoragent@cursor.com>
…finalize. Take upstream libveritas 0.3.3, prover Bearer/progress/opt-in calibrate, registry auth, and logs UI. Drop fork prover Basic auth, calibration disk cache, and API body-logging middleware. Co-authored-by: Cursor <cursoragent@cursor.com>
Certificate publish no longer waits for 150 on-chain confirmations. A confirmed commitment whose on-chain tip root matches is sufficient, which is what the pipeline already reported; the extra gate mostly stalled publishing for hours after the commitment was usable. Removes the SUBS_PUBLISH_REQUIRE_FINALIZED env var and the matching --publish-require-finalized flag, publish_blocked_reason() and publish_gate() in core, the require_finalized parameter on publish_certs(), the publish_require_finalized field on AppState, and the publish_require_finalized / publish_allowed / publish_blocked_reason fields on the pipeline API response. The 409 Conflict mapping in the publish route goes with them, since nothing can produce that state now. In the UI, the publish button and the bulk publish action no longer disable themselves or carry a blocked-reason tooltip. The 150-confirmation threshold still gates when the next local commit can start, so the Finalized pipeline step keeps its meaning; it just no longer blocks publish. Docs are updated to say so. Also adds a commented SUBS_BACKUP_DIR placeholder to .env.example. Co-authored-by: Cursor <cursoragent@cursor.com>
subs-* covers the release tarballs and zips produced when snapshotting a build; data_* covers the data_backup/ and data_bak/ directories holding copies of config.db and per-space subs.db. Neither belongs in the repo, and the .db copies contain live operator state. Co-authored-by: Cursor <cursoragent@cursor.com>
setup_subs_env.sh and setup_subs_local.sh hold real wallet and RPC credentials for a specific machine, so they should never be tracked. The pattern is lowercase on purpose: SETUP_NEW_SPACE.md is documentation and stays in the repo. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Closing this for now. It grew into a broad changeset spanning Docker/env configuration, auth, docs, and tooling, which makes it hard to review as a unit. I'll resubmit the pieces as smaller, focused PRs from dedicated branches. |
Author
|
I'm closing this for now so that I can sync cleanly. May open it again later. |
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.
Wire spaced RPC user/password from CLI into AppState so /rpc/spaced no longer uses hardcoded credentials, and add a clear startup console URL that includes the port.