chore: update to 3.9.3 and add small fixes for license selection#27485
Conversation
praveen-influx
left a comment
There was a problem hiding this comment.
Left a comment for clarification (or improvement), given it's for 3.9.3 it's probably ok to ship it.
| # Open URL in the user's default browser. Runs synchronously and surfaces | ||
| # errors so a failed launch is visible. |
There was a problem hiding this comment.
Do we need to do some "timeout" or something - if I ssh on to my box and run this command it'd possibly block
There was a problem hiding this comment.
It'll also probably fail from codex/claude/etc if it requires interactivity.
cannonpalms
left a comment
There was a problem hiding this comment.
Version bump to 3.9.3 is correct (both INFLUXDB_OSS_VERSION and INFLUXDB_ENT_VERSION, plus the help-text default). The accompanying cleanups look good: consolidating the three duplicated license prompts into prompt_license_type() (re-prompting on invalid input and aborting non-interactively rather than silently defaulting to trial is the safer behavior), the synchronous browser-open with error surfacing, the Explorer 80 → 8080 port fix for influxdb3-ui v1.7.0, and the STARTUP_CHOICE normalization. LGTM.
Updates to 3.9.3 and fixes issues on license selection (plus browser opening)