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

Anonymous access to graphql interface #861

Merged
merged 1 commit into from
May 14, 2024
Merged

Conversation

KaiVolland
Copy link
Member

@KaiVolland KaiVolland commented May 13, 2024

This adds anonymous acces to the graphql interface.

This is needed as the gisclient gets its layers via the /graphql interface. The graphql queries themeselves are still secured via the permission evaluators.

❗ BREAKING CHANGE: anonymous access to graphql interface

Description

Related issues or pull requests

Pull request type

  • Bugfix
  • Feature
  • Dependency updates
  • Tests
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe)

Do you introduce a breaking change?

  • Yes
  • No

Checklist

  • I understand and agree that the changes in this PR will be licensed under the
    Apache Licence Version 2.0.
  • I have followed the guidelines for contributing.
  • The proposed change fits to the content of the code of conduct.
  • I have added or updated tests and documentation, and the test suite passes (run mvn test locally).
  • I have added a screenshot/screencast to illustrate the visual output of my update.

BREAKING CHANGE: anonymous access to graphql interface
@KaiVolland KaiVolland marked this pull request as ready for review May 13, 2024 14:17
@KaiVolland KaiVolland merged commit 38b45cd into main May 14, 2024
4 checks passed
Copy link
Contributor

🎉 This PR is included in version 20.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants