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

Upgrade metric api #539

Merged
merged 6 commits into from
Sep 22, 2020
Merged

Upgrade metric api #539

merged 6 commits into from
Sep 22, 2020

Conversation

kaduartur
Copy link
Contributor

- What I did
I upgraded metrics API to improve security

- How to verify it
you can send a POST to https://ritchie-metrics.devdennis.zup.io/v2/metrics to see the authentication

- Description for the changelog
Upgrade metric API

Signed-off-by: Kadu Artur Prussek <kadu.artur@gmail.com>
Signed-off-by: Kadu Artur Prussek <kadu.artur@gmail.com>
@kaduartur kaduartur added 🔨 improvement Improvement in features ✔️ ready-for-review ready for review labels Sep 21, 2020
@kaduartur kaduartur added this to the 2.0.6 milestone Sep 21, 2020
@kaduartur kaduartur self-assigned this Sep 21, 2020
@codecov-commenter
Copy link

codecov-commenter commented Sep 21, 2020

Codecov Report

Merging #539 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #539   +/-   ##
=======================================
  Coverage   78.51%   78.51%           
=======================================
  Files          98       98           
  Lines        3169     3169           
=======================================
  Hits         2488     2488           
  Misses        497      497           
  Partials      184      184           

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 acfbd99...bb03056. Read the comment docs.

@kaduartur
Copy link
Contributor Author

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Sep 21, 2020

👌 Merged branch feature/update_metric_api into qa

Signed-off-by: Kadu Artur Prussek <kadu.artur@gmail.com>
@kaduartur
Copy link
Contributor Author

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Sep 21, 2020

👌 Merged branch feature/update_metric_api into qa

Signed-off-by: Kadu Artur Prussek <kadu.artur@gmail.com>
@kaduartur
Copy link
Contributor Author

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Sep 21, 2020

👌 Merged branch feature/update_metric_api into qa

Signed-off-by: Kadu Artur Prussek <kadu.artur@gmail.com>
@kaduartur
Copy link
Contributor Author

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Sep 21, 2020

👌 Merged branch feature/update_metric_api into qa

@kaduartur kaduartur merged commit 5fb596c into ZupIT:master Sep 22, 2020
@kaduartur kaduartur deleted the feature/update_metric_api branch September 22, 2020 12:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🔨 improvement Improvement in features ✔️ ready-for-review ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants