-
Notifications
You must be signed in to change notification settings - Fork 7
v1.0.0 #107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
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
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Still needs work on the README.md to reflect changes. |
…p customization feat(docker): create 'photon' user and group with specified PUID/PGID in Dockerfile feat(docker): add gosu to Dockerfile for running commands as specific user fix(permissions): update ownership and permissions logic to use 'photon' user/group refactor(start-photon.sh): remove unused ES_UID and ES_GID variables refactor(start-photon.sh): centralize user/group setup into `setup_user` function refactor(start-photon.sh): use `gosu` to run photon as the 'photon' user refactor(start-photon.sh): ensure data directory permissions are set before starting photon
Reorganize the "Configuration Options" section to appear after "Example Docker Compose" for better flow. Add `PUID` and `PGID` environment variables to the configuration options table.
fix(Dockerfile): set correct permissions for photon.jar and /photon directory refactor(start-photon.sh): remove redundant chmod 755 command refactor(start-photon.sh): simplify md5sum verification refactor(start-photon.sh): simplify tar extraction refactor(start-photon.sh): simplify elasticsearch directory removal refactor(start-photon.sh): simplify download logic refactor(start-photon.sh): simplify md5 verification refactor(start-photon.sh): simplify archive extraction feat(start-photon.sh): add debug log for current owner feat(start-photon.sh): add chown for temp directory
chore(config.sh): add chown command to ensure correct permissions for /photon directory
The photon service was previously started as the root user, which is not a good security practice. This commit changes the service to run as the 'photon' user using `gosu`, improving security.
feat(Dockerfile): add python3.12 and uv to the image refactor(Dockerfile): change entrypoint to use uv run main.py
…ting, and type-checking steps
refactor: improve disk space checks and logging for sequential updates
…echeck, and vulture
…rphaned process cleanup
…d remove old index
Now uses a marker file instead of checking the actual index file. Extracted index directories will not have the same timestamp as the remote download file.
Replaces `COUNTRY_CODE` with `REGION` for more flexible data selection, supporting planet, continent, and specific sub-region downloads.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Versioning Change
The versioning for this Docker image has been decoupled from the upstream Photon version. This allows for more intuitive versioning that reflects changes to the container environment and scripts, rather than centering around changes to the underlying Photon application.
Breaking Changes
The data volume path has been changed from
/photon/photon_data
to/photon/data
.docker-compose.yml
update:COUNTRY_CODE replaced with REGION system
Old: COUNTRY_CODE=de
New: REGION=andorra or REGION=europe
Default base URL changed to https://r2.koalasec.org/public (was /public/experimental)
Index file naming changed to include OpenSearch version (e.g., photon-db-planet-0.7OS-latest.tar.bz2)
New Features
photon
) for improved security. The user and group IDs can be set viaPUID
andPGID
environment variables to manage volume permissions.APPRISE_URLS
environment variable. feat: Add Apprise notifications for update status #100HEALTHCHECK
instruction has been added to the Dockerfile. The container will show asunhealthy
whenever the photon-api is not available.Improvements
beta
) and stable (latest
) tags. A workflow for code quality checks usingruff
,ty
andvulture
was added.