Skip to content
This repository has been archived by the owner on Dec 6, 2022. It is now read-only.

#21: Generate JWT tokens for API development #22

Merged
merged 4 commits into from
Aug 26, 2020

Conversation

th0mas
Copy link
Collaborator

@th0mas th0mas commented Aug 26, 2020

Fixes #21.


  • Create helper function to generate JWTs
  • Create a mix task so we can access from the command line
  • Write tests for new functionality

@th0mas th0mas self-assigned this Aug 26, 2020
@codecov
Copy link

codecov bot commented Aug 26, 2020

Codecov Report

Merging #22 into master will increase coverage by 1.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
+ Coverage   56.93%   57.97%   +1.04%     
==========================================
  Files          24       26       +2     
  Lines         202      207       +5     
==========================================
+ Hits          115      120       +5     
  Misses         87       87              
Impacted Files Coverage Δ
lib/mix/tasks/smart_home/gen_token.ex 100.00% <100.00%> (ø)
lib/smart_home_auth.ex 100.00% <100.00%> (ø)

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 6e2f5d1...b9388f3. Read the comment docs.

@th0mas th0mas assigned nelsonic and unassigned th0mas Aug 26, 2020
@th0mas th0mas marked this pull request as ready for review August 26, 2020 14:05
lib/smart_home_auth.ex Outdated Show resolved Hide resolved
Copy link
Member

@nelsonic nelsonic left a comment

Choose a reason for hiding this comment

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

@th0mas looks good. 👍

@nelsonic nelsonic merged commit d329046 into master Aug 26, 2020
@nelsonic nelsonic deleted the 21-generate-jwt-locally branch August 26, 2020 14:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generate JWT tokens locally for development
2 participants