Skip to content

[v16] Prevent auth from starting when an incompatible major upgrade is detected#44360

Merged
vapopov merged 1 commit intobranch/v16from
vapopov/major-version-check-v16
Jul 17, 2024
Merged

[v16] Prevent auth from starting when an incompatible major upgrade is detected#44360
vapopov merged 1 commit intobranch/v16from
vapopov/major-version-check-v16

Conversation

@vapopov
Copy link
Copy Markdown
Contributor

@vapopov vapopov commented Jul 17, 2024

Backport #43520 to branch/v16

…cted (#43520)

* Teleport auth should refuse to start when jumping major versions (#41782)

* linter fixes

* code review changes:
- replaced to use process storage and state itme
- regexp replaced with semver
- additional check for v18 for empty version record in storage

* code review changes:
- add integration test
- store latest version in separate key

* code review changes:
- table tests
- error message

* code review changes:
- minor changes

* code review changes:
- reuse context for write operation

* code review changes:
- simplify integration test
- add downgrade check
- minor changes

* code review changes:
- move logic to the auth init

* code review changes:
- add development env to skip version check
- init the process storage if it is not set

* code review changes:
- add interface for local process storage
- use semver in storage instead

* change interface to version storage

* add fake version storage for other tests
@vapopov vapopov added backport no-changelog Indicates that a PR does not require a changelog entry labels Jul 17, 2024
@vapopov vapopov enabled auto-merge July 17, 2024 21:56
@vapopov vapopov added this pull request to the merge queue Jul 17, 2024
Merged via the queue into branch/v16 with commit eefc416 Jul 17, 2024
@vapopov vapopov deleted the vapopov/major-version-check-v16 branch July 17, 2024 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport no-changelog Indicates that a PR does not require a changelog entry size/md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants