Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

when new a style, clustering permission is 640,how to change to 644? #687

Open
windcatcher opened this issue Aug 13, 2024 · 4 comments
Open

Comments

@windcatcher
Copy link

windcatcher commented Aug 13, 2024

What is the bug or the crash?

when new a style, clustering permission is 640,how to change to 644?

  • the docker image version is kartoza/geoserver:2.24.0
  • the docker run script is
docker run --name "cluster-geoserver" --restart=on-failure  -p 8081:8080  --privileged=true -e CLUSTERING=true   -e CLUSTER_DURABILITY=false -e DB_BACKEND=POSTGRES   -e HOST=host -e POSTGRES_PORT=5432  -e POSTGRES_DB=gis1 -e POSTGRES_USER=docker123e -e POSTGRES_PASS=docker1@123e -e SSL_MODE=ALLOW -e  BROKER_URL=failover:\(tcp://host1:61616,tcp://host2:61616,tcp://host3:61616\)  -e READONLY=enabled -e TOGGLE_MASTER=false -e TOGGLE_SLAVE=true -e EMBEDDED_BROKER=disabled -e  RUN_AS_ROOT=TRUE -e  GEOSERVER_ADMIN_USER=admin -e GEOSERVER_ADMIN_PASSWORD=reerwqe -e STABLE_EXTENSIONS= -e COMMUNITY_EXTENSIONS=jms-cluster-plugin -v  /tgis/cluster/geoserver_data:/opt/geoserver/data_dir -d -t kartoza/geoserver:2.24.0
  • the new style [test.sld] permission is 640

a5dcc9a595cb3969a7c8c4efea90d61

Steps to reproduce the issue

style file permission

Versions

kartoza/geoserver:2.24.0

Additional context

@NyakudyaA
No response

@NyakudyaA
Copy link
Collaborator

@windcatcher I will take a look and revert

@windcatcher
Copy link
Author

windcatcher commented Dec 15, 2024

Can the problem be reproduced? @NyakudyaA

@NyakudyaA
Copy link
Collaborator

Can the problem be reproduced? @NyakudyaA

I haven't had a chance to test, I will get onto it during the holiday season. If you feel like you need this to be investigated with priority please look at the support level agreements

@NyakudyaA
Copy link
Collaborator

@windcatcher I see all the files are currently with permission 640, What would be the advantage of making the files 644, I looked at the code and it seems like we need to add some logic inside https://github.com/kartoza/docker-geoserver/blob/develop/scripts/functions.sh#L478, So should all the files created inside the data dir be 644 or just the styles ?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants