-
Notifications
You must be signed in to change notification settings - Fork 144
The admin app moved to the admin. subdomain instead of the admin/ path
#244
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
Open
namedgraph
wants to merge
35
commits into
master
Choose a base branch
from
develop
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
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
… path (#243) * The admin app moved to the `admin.` subdomain instead of the `admin/` path `ldh:origin` replaces `ldt:base` in config * `RemoveIpValve` config params Also fixed `Connector` proxy params * Separated Docker build and run stages in the CI * CI debug logs * Debug CI * Debug * Debug CI * Hardcoded `${HTTP_PORT}` value in `HEALTHCHECK` * atomgraph/letsencrypt-tomcat base image bump * Debug server.xml * Replaced `ldh:base` with `ldh:origin` Moved RDF data with relative URIs out from admin.trig Removed origin output * Fixed `ADMIN_BASE_URL` * Undo http-tests workflow chnages * `ADMIN_BASE_URL` fix in HTTP test * Replacing the last `ldt:base` usages in XSLT with `ldh:origin` `make-public` CLI script fix * Whitespace * Javadoc * New `Application::normalizeOrigin` method
* Updated ACL queries to only retrieve authorizations for the current dataspace That is done by injecting `$base` into the queries and adding `FILTER(strstarts(str(?g), str($base)))` * Adding fake cross-dataspace authorizations to test dataset * Throw `NotFoundException` when non dataspace (application) is matched Inject `Optional<Application>` instead of just `Application` * Set `-f` on curl calls in tests * Refactor agent metadata and authorization handling Introduces separate templates for owner and secretary authorizations, updates entrypoint.sh to extract and manage agent metadata and authorizations per app, and adds a SPARQL query for agent metadata extraction. Updates Dockerfile and test configurations to support new dataset and metadata handling. Refactors Java `ValidatingModelProvider` to improve application presence checks and mbox processing. Also improves XSLT title rendering and test scripts for non-existent dataspaces. * Test fix * Error flags in CLI scripts
Entrypoint cleanup
Response filter that adds CORS (Cross-Origin Resource Sharing) headers to allow cross-origin access. Closes #73
Excluded the indirect Xerces dependency and now the JDK internal Xerces is the default XML parser
* Validate whether IP address is internal in `ProxyResourceBase` Added new `ForbiddenExceptionMapper` * New tests for IP address blocking * Allow `localhost` proxying
Closes #253
`Content-Security-Policy` response header is now set when serving uploaded files
…d localhost The filter now uses baseURI.getHost() to determine which URLs to rewrite, allowing it to work correctly on production deployments (e.g., linkeddatahub.com) instead of only on localhost. This fixes a performance regression introduced in the admin subdomain refactor where internal requests would go through external network instead of Docker network. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Also added HTTP tests
Removed debug output from entrypoint
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.
No description provided.