Skip to content

Conversation

@mrsaicharan1
Copy link
Member

Fixes #6430

Short description of what this resolves:

  • Added JWT headers to ensure a proper workflow which is only functional for authenticated users.

Changes proposed in this pull request:

  • Added JWT required headers

@auto-label auto-label bot added the feature label Aug 30, 2019
@codecov
Copy link

codecov bot commented Aug 30, 2019

Codecov Report

Merging #6431 into development will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #6431      +/-   ##
===============================================
+ Coverage         64.4%   64.41%   +<.01%     
===============================================
  Files              288      288              
  Lines            15056    15058       +2     
===============================================
+ Hits              9697     9699       +2     
  Misses            5359     5359
Impacted Files Coverage Δ
app/api/orders.py 26.61% <100%> (+0.36%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 46c5fdf...a0afece. Read the comment docs.

@mrsaicharan1
Copy link
Member Author

@iamareebjamal @uds5501 @shreyanshdwivedi Please review

@iamareebjamal iamareebjamal merged commit b1fd647 into fossasia:development Aug 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AliPay/Omise routes should only be accessible for authenticated users

4 participants