Skip to content
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

Update lizmapServices.class.php #5369

Merged
merged 2 commits into from
Feb 6, 2025
Merged

Update lizmapServices.class.php #5369

merged 2 commits into from
Feb 6, 2025

Conversation

Antoviscomi
Copy link
Contributor

@Antoviscomi Antoviscomi commented Feb 6, 2025

Causes of bad url construction such as http://myhost/cgi-bin/qgis_mapserv.fcgi/lizmap//server.json adding a double slash before of server.json and avoid the correct recognization of Lizmap server plugin on Master

Causes of bad url construction such as http://myhost/cgi-bin/qgis_mapserv.fcgi/lizmap//server.json adding a double slash before of server.json and avoid the correct recognization of Lizmap server plugin on Master
@github-actions github-actions bot added this to the 3.10.0 milestone Feb 6, 2025
@Gustry
Copy link
Member

Gustry commented Feb 6, 2025

It's due to my PR #5266 on 3.9 and master

Copy link
Member

@Gustry Gustry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution.

We need to check with #5262

lizmap/modules/lizmap/classes/lizmapServices.class.php Outdated Show resolved Hide resolved
Copy link
Contributor Author

@Antoviscomi Antoviscomi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix Causes of bad url construction such as http://myhost/cgi-bin/qgis_mapserv.fcgi/lizmap//server.json adding a double slash before of server.json and avoid the correct recognization of Lizmap server plugin on Master

@Gustry Gustry merged commit e2fd46d into 3liz:master Feb 6, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants