asgardeo / asgardeo/asgardeo-java-oidc-sdk

Add missing validations for the REQUIRED configurations.

Open
#14 0 comments 0 reactions 0 assignees View on GitHub
Priority/High Severity/Critical
Dominant language
Java
Stars
18
Forks
28
PR merge metrics
No merged PRs in 30d

Description

**Description:**
In the Configurations catalog added with https://github.com/asgardio/asgardio-tomcat-oidc-agent/pull/1/commits/0ec2633adb762bf76cee7252e206912c38cb3e59, some properties are marked as REQUIRED in the app.properties file.
In the config provider or in the DefaultOIDCManager constructor, we need to validate these properties that are marked as REQUIRED.

As of now, validations for the following properties are implemented.

- Consumer Key / Client ID
- Consumer Secret / Client Secret
- Callback URI / Redirect URI
- Scope

Missing validations for the other REQUIRED parameters must also be added there.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the REQUIRED entries in app.properties and the existing validation for Consumer Key, Consumer Secret, Callback URI, and Scope. Then inspect the config provider and DefaultOIDCManager constructor to identify the remaining REQUIRED parameters; done means each is validated consistently with the existing checks.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.