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

[DNM] Ona Changes #49

Open
wants to merge 47 commits into
base: master
Choose a base branch
from
Open

[DNM] Ona Changes #49

wants to merge 47 commits into from

Conversation

DavisRayM
Copy link

@DavisRayM DavisRayM commented Sep 30, 2022

This PR is just for viewing purposes; The branch attached to the PR is the new proposed main branch for the fork.

The branch is synced with the release 2.022.24b on kobotoolbox/kpi

@DavisRayM DavisRayM changed the base branch from ona-custom-changes to master September 30, 2022 14:01
@DavisRayM DavisRayM force-pushed the ona-changes branch 2 times, most recently from 2b1e752 to 248e00f Compare September 30, 2022 15:23
DavisRayM and others added 22 commits October 11, 2022 11:21
- Stops users from being able to modify OnaData controlled resources on
  Kobo home...
- Use Asset name as id_string
- Add `published_by_formbuilder` to the deployment payload
- Add JWTAuthentication class
- Add Database router that stops writes and migrations to Kobocat
Signed-off-by: Kipchirchir Sigei <[email protected]>
- Add management command
model = Token

def authenticate(self, request):
cookie_jwt = request.COOKIES.get(settings.KPI_COOKIE_NAME)
Copy link
Member

Choose a reason for hiding this comment

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

would it be possible to provide an alternative to cookie authentication? perhaps place the same value supplied in this cookie in an Authorization header?

Copy link
Author

Choose a reason for hiding this comment

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

You can use the ONADATA_KOBOCAT_AUTH header

Signed-off-by: Kipchirchir Sigei <[email protected]>
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.

5 participants