Skip to content

sc-158714: Python 3 migration, security fixes, dep updates#8

Open
ptamb3 wants to merge 2 commits into
openalpr:masterfrom
ptamb3:sc-158714
Open

sc-158714: Python 3 migration, security fixes, dep updates#8
ptamb3 wants to merge 2 commits into
openalpr:masterfrom
ptamb3:sc-158714

Conversation

@ptamb3
Copy link
Copy Markdown

@ptamb3 ptamb3 commented May 7, 2026

Summary

  • Upgrade daemon Dockerfile from Ubuntu 18.04 (EOL) to 22.04
  • Fix insecure HTTP -> HTTPS for GPG key download (MITM risk)
  • Migrate webservice.py from Python 2 to Python 3 syntax
  • Replace time.clock() (removed in 3.8) with time.perf_counter()
  • Replace bare except: with Exception + error logging
  • Fix deprecated threading.currentThread() -> current_thread()
  • Update requirements: remove Python 2 backports, modernize deps
  • Update webservice Dockerfile to python3

Test plan

  • Build daemon Docker image
  • Build webservice Docker image
  • Test webservice /v1/identify/plate endpoint
  • Verify timing debug output works

Shortcut: https://app.shortcut.com/rekor/story/158714

🤖 Generated with Claude Code

ptamb3 and others added 2 commits May 7, 2026 13:45
- Upgrade daemon Dockerfile from Ubuntu 18.04 to 22.04
- Fix insecure HTTP -> HTTPS for GPG key download
- Replace deprecated MAINTAINER with LABEL
- Migrate webservice.py: print statements, time.clock -> perf_counter
- Replace bare except with Exception + logging
- Fix deprecated threading.currentThread -> current_thread
- Update requirements: remove Python 2 backports, modernize deps
- Update webservice Dockerfile to python3

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Dockerfile.nvidia: CUDA 8.0/Ubuntu 16.04 -> CUDA 10.2/Ubuntu 18.04 (JetPack 4.6)
- Dockerfile.nvidia10: CUDA 10.0/Ubuntu 18.04 -> CUDA 12.6/Ubuntu 22.04 (JetPack 6.2.2)
- Fix insecure HTTP -> HTTPS for GPG key download in both files
- Replace deprecated MAINTAINER with LABEL
- Add --no-install-recommends to reduce image size

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant