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.

Sign up for GitHub

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

Jump to bottom

Feat/myaia #24

Closed
wants to merge 37 commits into from
Closed

Feat/myaia #24

wants to merge 37 commits into from

Conversation

govind-sg
Copy link

No description provided.

@dryice
incoorperate the middleware PR:
57ad700 
manosim@813739f
@dryice
use correct middleware_class name
96741a2
@dryice
add api_key object to the request so other parts can read
90d87e9
@dryice
update the middleware to use Django 1.8.x
26cd1a1
@dryice
refactor to send url path for checking
6473b21
@dryice
support path check for given API key
709c8e3
@dryice
catch up the change in last commit
11c62c2
@dryice
don't request api_key for http OPTIONS
48ad357
@dryice
only check api-key when "/api/" in the URL path
5afa9c7
@dryice
show path_re in admin
cd3a295
@dryice
skip api-key check to be backward compitable
f284f90
@dryice
update migration so all url is passed
714073e
@dryice
a bit doc
bffdf01
@dryice
don't require api-key for mobilepulsa callback
7697011
@dryice
add a new "KeyOwner" model
ddb6adb
@dryice
if an api-key is supplied, check it, even if the user_agent says it c…
e1e3598 
…an be skipped
@dryice
allow empty owner for migration
9cae57a
@dryice
update tox to test on Django 1.8 and 1.11 that we are using and will use
a1f2982
@dryice
autopep8
14bea4f
@dryice
add test with api key owner
936b91f
@dryice
we are using DRF 3.1.3, so test against it
1dd5b76
@dryice
optimize code, don't query DB unless necessary
73853e7
@dryice
don't check api-key if it's from the ELB healthy checker
29f9a45
@dryice
bug fix
3e89678
@dryice
generate html coverage report
e6c2087
@dryice
basic project coding setup
2854265
@dryice
git ignore the coverage report
6e1be6a
@dryice
fix test setup: use the correct settings name
7d19a24
@dryice
now we get 100% test coverage on both Django 1.8 and 1.11 :)
f6a5df2
@dryice
if mobile send an api-key, fetch it even when we can skip the check
e41f368
@govind-sg govind-sg closed this Dec 2, 2021
@govind-sg govind-sg deleted the feat/myaia branch December 2, 2021 13:10
@govind-sg govind-sg restored the feat/myaia branch December 2, 2021 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@govind-sg @dryice