Skip to content
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

[WIP][RW-969] Post API - api key per user #812

Open
wants to merge 2 commits into
base: feature/RW-831-post-api
Choose a base branch
from

Conversation

attiks
Copy link
Contributor

@attiks attiks commented May 16, 2024

Still WIP

  1. Add a "trusted user" reference field to the provider entity type (filled by the RW team after verification/agreement etc. with partner).
  2. Add an API key field to users
  3. Add a user route for a user to get a new automatically generated key (view once)
  4. Add a user route for a user to see the providers they are trusted for (so they can get the provider UUID notably)
  5. Update the provider entity to validate the API key against the API keys of the trusted users

@attiks attiks requested a review from orakili May 16, 2024 15:26
Copy link

Coverage Report

Totals Coverage
Statements: 15.32% ( 2123 / 13859 )
Methods: 19.83% ( 162 / 817 )
Lines: 15.04% ( 1961 / 13042 )

Copy link

Build output

Composer Validate success

PHP Lint success

Docker Build success

Environment Setup success

Site Install success

PHP Code Sniffer success

Software Versions PHP 8.2.19 (cli) (built: May 13 2024 18:20:15) (NTS) Copyright (c) The PHP Group Zend Engine v4.2.19, Copyright (c) Zend Technologies with Zend OPcache v8.2.19, Copyright (c), by Zend Technologies with Xdebug v3.3.2, Copyright (c) 2002-2024, by Derick Rethans Composer version 2.7.2 2024-03-11 17:12:18
Drupal Logs

Pusher: @attiks, Action: pull_request, Workflow: Run tests

Copy link
Collaborator

@orakili orakili left a comment

Choose a reason for hiding this comment

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

Looks good, thanks.

@orakili orakili changed the title Rw 969 [WIP][RW-969] Post API - api key per user Aug 5, 2024
@orakili orakili added the blocked This PR is blocked, do not merge it label Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked This PR is blocked, do not merge it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants