Skip to content

DEV-683 Implement OIDC for GeoServer#27

Merged
jason-raitz merged 10 commits into
mainfrom
DEV-683_OIDC
Jul 13, 2026
Merged

DEV-683 Implement OIDC for GeoServer#27
jason-raitz merged 10 commits into
mainfrom
DEV-683_OIDC

Conversation

@jason-raitz

@jason-raitz jason-raitz commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Purpose

Since we've done away with the local geoserver repo, this is a way to do development on geoserver and geoserver-secure locally.

This repo uses keycloak to mimic CalNet OIDC. There are also some changes to the README to reflect one time adding of oidc configuration to the geoserver and geoserver-secure containers.

@jason-raitz jason-raitz self-assigned this Jul 6, 2026

@anarchivist anarchivist left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks good; i have some questions about whether we can incorporate the oidc config for geoserver into something called through the compose file.

also, maybe out of scope for this PR, but i'n now wondering about whether we should be building our own keycloak image.

Comment thread data/geoserver/geoserver_oidc_seed/security/filter/keycloak/config.xml Outdated
Comment thread data/geoserver/geoserver_oidc_seed/security/config.xml Outdated
Comment thread bin/add-keycloak-to-geoserver.sh Outdated

@awilfox awilfox left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Some feedback and suggestions.

Comment thread bin/add-keycloak-to-geoserver.sh Outdated
Comment thread bin/add-keycloak-to-geoserver.sh Outdated
Comment thread bin/add-keycloak-to-geoserver.sh Outdated
Comment thread bin/add-keycloak-to-geoserver.sh Outdated
Comment thread bin/add-keycloak-to-geoserver.sh Outdated
Comment thread bin/add-keycloak-to-geoserver.sh Outdated
Comment thread bin/geoserver-oidc-config.sh Outdated
Comment thread docker-compose.yml
Comment thread docker-compose.yml Outdated
Comment thread env.example Outdated
 - also volume mount fix for postgres 18 image
@jason-raitz jason-raitz marked this pull request as ready for review July 8, 2026 19:20

@awilfox awilfox left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me, but I don't have the cycles to spin it up locally for testing so I don't want to explicitly approve it.

@yzhoubk

yzhoubk commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

@jason-raitz

When entering the restricted GeoServer URL http://localhost:8081/geoserver in browser, it is redirected to public GeoServer URL http://localhost:8080/geoserver. Could you check this in your local?

@anarchivist anarchivist left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

i'm seeing the same thing as @yzhoubk - i think we might need separate client configs and geoserver OIDC configs for each geoserver instance.

Comment thread data/geoserver/keycloak/config/berkeley-local.yaml
Comment thread bin/geoserver-oidc-config.sh
@jason-raitz

Copy link
Copy Markdown
Contributor Author

@jason-raitz

When entering the restricted GeoServer URL http://localhost:8081/geoserver in browser, it is redirected to public GeoServer URL http://localhost:8080/geoserver. Could you check this in your local?

Great catch @yzhoubk! Thanks.

 - added 2 new env vars for geoserver-secure
 - added a separate client for geoserver-secure within the keycloak realm
 - adjusted host and docker hostnames and ports for configs
@yzhoubk

yzhoubk commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Restricted GeoServer looks good. Map services are created on both Restricted and public Geoserver.

@anarchivist anarchivist self-requested a review July 13, 2026 17:48

@anarchivist anarchivist left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

r+. thanks!

@jason-raitz jason-raitz merged commit 2547c74 into main Jul 13, 2026
5 checks passed
@jason-raitz jason-raitz deleted the DEV-683_OIDC branch July 13, 2026 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants