Change WDK guest user creation mechanism
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 20/100
- Issue type
- Feature
- Clarity
- Needs clarification
- Activity status
- Stale
- Tech stack
- java
- Domain
- authentication, backend-api-design
Research direction
No files, tests, or entry points are named. Start by locating the Jersey filter that creates the persistent guest user and compare it with the newer service architecture. The issue needs an agreed design for endpoint-specific user requirements and the possible bearer-token model before implementation can be considered done.
Written by the indexing model from the issue text.
Description
WDK has a Jersey filter that ensures there is always a user present for service request methods to use. If no registered user is logged in, this serves as a persistent guest user (attached to the session). Since the conversion to a client/service architecture, this method of providing users is overkill since not all services (in fact, many services) require a user or are user-agnostic (i.e. could use the "system user".
We should rework WDK to work more like our new service architecture, letting specific endpoints declare whether they need a user or not. This issue is to track ideas around this concept and may be related to the goal of factoring user authentication into a bearer token model, where WDK would simply be one of many services that respect VEuPathDB bearer tokens. It may even be a contributing reason to move WDK to the new service framework.
- Dominant language
- Java
- Stars
- 1
- Forks
- 4
- Avg merge
- 1m
- Merged PRs (30d)
- 2
Getting set up
- Ships a Dockerfile or Docker Compose file
- No pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from VEuPathDB/WDK
-
Difficulty 3/5 1-2 days Newbie friendliness 62/100
-
Difficulty 4/5 3-5 days Newbie friendliness 38/100
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
-
Feature request: Add redirectUrl query parameter support to `/service/logout` endpointPossibly taken A pull request linked to this issue is open or already merged. Openenhancement
Difficulty 3/5 1-2 days Newbie friendliness 42/100
-
Refine login API to get rid of cookies and automatic guestsPossibly taken @ryanrdoherty claimed this 687 days ago. Open
Similar issues
-
Flight SQL JDBC parses boolean properties case-sensitively, silently disabling TLS for useEncryption=TRUEPossibly taken @Arawoof06 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 79/100
apache/arrow-java#1357 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
apache/rocketmq-dashboard#6610 ·
Maintainers usually reply within 3 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
MetricsHub/metricshub-community#1396 ·
Maintainers usually reply within 1 day
-
L: java:gradle L: java:maven L: ruby:bundler T: bug 🐞
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
dependabot/dependabot-core#16547 ·
Maintainers usually reply within 1 day
-
Citationcount column sorts alphabetically instead of numericallyPossibly taken @kpunia306 claimed this today. Openstatus: needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
JabRef/jabref#17459 · 4 comments ·
Maintainers usually reply within 1 day