add an example of basic user user and group logic#3799
Draft
OyvindLGjesdal wants to merge 1 commit intoapache:mainfrom
Draft
add an example of basic user user and group logic#3799OyvindLGjesdal wants to merge 1 commit intoapache:mainfrom
OyvindLGjesdal wants to merge 1 commit intoapache:mainfrom
Conversation
Aklakan
reviewed
Mar 15, 2026
| attemptByLocalhost(server, httpClient, dsname); | ||
| // and a SPARQL query | ||
| QueryExecHTTP.service(URL).httpClient(httpClient).query("ASK{}").ask(); | ||
| QueryExecHTTP.service(URL).httpClient(httpClient).query("ASK{}"); |
Contributor
There was a problem hiding this comment.
Without the .ask(), this line becomes a no-op - so it doesn't look right.
Contributor
Author
There was a problem hiding this comment.
Thanks, fixed. Also noticed an additional error (that the test that I expected to fail passed, so I had messed up the expect value).
Contributor
Author
|
Sorry for the messy I PR, I see that it is kind of verbose and a wip with commented sections, but thought it was usable as a demonstration for the docs update. I think at least it fails correctly now e.g |
* add failing test for demo purpose
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
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.
GitHub issue resolved #
Pull request Description:
Adds tests that shows that current configuration stanzas allows other authenticated users access to configured paths, for demonstration. Also adds a second test that tries the same using groups, which works.
How should the ini files look instead if examples and default (with commented example that has this behaviour) look?
Not for merging currently, but for demonstration.
The existing test access_userPassword already has a shiro where there are two users and paths are different, so the test can be changed there, by adding an assertion that expects user1 to not have ping access
https://github.com/apache/jena/blob/main/jena-fuseki2/jena-fuseki-main/testing/Shiro/shiro_userpassword.ini
By submitting this pull request, I acknowledge that I am making a contribution to the Apache Software Foundation under the terms and conditions of the Contributor's Agreement.
See the Apache Jena "Contributing" guide.