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

MF-1103 - API key should ignore empty expiration time #1104

Merged
merged 2 commits into from
Apr 7, 2020

Conversation

dborovcanin
Copy link
Collaborator

@dborovcanin dborovcanin commented Apr 7, 2020

Signed-off-by: Dušan Borovčanin dusan.borovcanin@mainflux.com

What does this do?

This pull request fixes API key expiration validation.

Which issue(s) does this PR fix/relate to?

This pull request fixes #1103.

List any changes that modify/break current functionality

API key validation should allow using tokens with an indefinite duration.

Have you included tests for your changes?

Yes.

Did you document any new/modified functionality?

Yes, API docs are updated.

Signed-off-by: Dušan Borovčanin <dusan.borovcanin@mainflux.com>
Signed-off-by: Dušan Borovčanin <dusan.borovcanin@mainflux.com>
@codecov-io
Copy link

Codecov Report

Merging #1104 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1104   +/-   ##
=======================================
  Coverage   77.13%   77.13%           
=======================================
  Files          95       95           
  Lines        6769     6771    +2     
=======================================
+ Hits         5221     5223    +2     
  Misses       1211     1211           
  Partials      337      337           
Impacted Files Coverage Δ
authn/keys.go 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 6c39c85...d2a3a97. Read the comment docs.

@dborovcanin dborovcanin merged commit 57f7ee2 into absmach:master Apr 7, 2020
manuio pushed a commit that referenced this pull request Oct 12, 2020
* Fix API keys expiration validation

Signed-off-by: Dušan Borovčanin <dusan.borovcanin@mainflux.com>

* Fix API docs

Signed-off-by: Dušan Borovčanin <dusan.borovcanin@mainflux.com>
@dborovcanin dborovcanin deleted the MF-1103 branch February 17, 2021 09:07
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.

API key should ignore empty expiration time
4 participants