Skip to content
This repository was archived by the owner on Jun 21, 2022. It is now read-only.

SAAS-175 Auth API#420

Merged
AlekSi merged 51 commits intoPMM-2.0from
SAAS-175-auth-api
Jul 28, 2020
Merged

SAAS-175 Auth API#420
AlekSi merged 51 commits intoPMM-2.0from
SAAS-175-auth-api

Conversation

@artemgavrilov
Copy link
Contributor

@artemgavrilov artemgavrilov commented Jul 14, 2020

Feature build: Percona-Lab/pmm-submodules#967

Integration with Percona Auth Service, added three server API methods SignUp and SingIn accessible only for grafana admin.

@artemgavrilov artemgavrilov changed the base branch from master to PMM-2.0 July 14, 2020 20:02
@codecov
Copy link

codecov bot commented Jul 14, 2020

Codecov Report

Merging #420 into PMM-2.0 will decrease coverage by 0.07%.
The diff coverage is 50.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           PMM-2.0     #420      +/-   ##
===========================================
- Coverage    49.93%   49.85%   -0.08%     
===========================================
  Files           90       91       +1     
  Lines        10405    10556     +151     
===========================================
+ Hits          5196     5263      +67     
- Misses        4810     4880      +70     
- Partials       399      413      +14     
Flag Coverage Δ
#all 48.97% <50.00%> (-0.02%) ⬇️
#cover 50.00% <48.40%> (-0.07%) ⬇️
#crosscover 49.85% <50.00%> (-0.08%) ⬇️
#update 21.80% <0.00%> (-0.32%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
main.go 0.00% <0.00%> (ø)
models/settings.go 100.00% <ø> (ø)
services/grafana/auth_server.go 60.38% <ø> (ø)
services/server/server.go 12.67% <6.25%> (-0.26%) ⬇️
services/platform/platform.go 49.09% <49.09%> (ø)
models/settings_helpers.go 80.11% <91.30%> (+1.63%) ⬆️
utils/testdb/db.go 100.00% <100.00%> (ø)
services/supervisord/maintail.go 84.21% <0.00%> (-15.79%) ⬇️
services/supervisord/supervisord.go 65.24% <0.00%> (-0.66%) ⬇️

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 95ba56e...e7a26b9. Read the comment docs.

@artemgavrilov artemgavrilov marked this pull request as ready for review July 16, 2020 16:50
@artemgavrilov
Copy link
Contributor Author

Feature build in progress

@artemgavrilov artemgavrilov requested a review from palash25 July 16, 2020 16:51
@artemgavrilov
Copy link
Contributor Author

Something strange happening with CI.

@AlekSi AlekSi self-assigned this Jul 27, 2020
@artemgavrilov artemgavrilov changed the title SAAS-175 Auth API SAAS-175 SAAS-174 Auth API Jul 27, 2020
@artemgavrilov artemgavrilov changed the title SAAS-175 SAAS-174 Auth API SAAS-175 Auth API Jul 28, 2020
@AlekSi AlekSi merged commit 69a548f into PMM-2.0 Jul 28, 2020
@AlekSi AlekSi deleted the SAAS-175-auth-api branch July 28, 2020 16:29
askomorokhov pushed a commit that referenced this pull request Mar 22, 2021
* SAAS-175 Update saas dependency

* SAAS-175 Update pmm dependency

* SAAS-175 Implement authentication methods

* SAAS-175 Update pmm dependency, remove RefreshSession API method

* SAAS-175 Add auth tests

* SAAS-175 Refactoring

* SAAS-175 Fix tests

* SAAS-175 Mute linter

* SAAS-175 Refactoring

* SAAS-175 Update pmm dependency, fix build

* SAAS-175 Refactoring

* SAAS-175 Refactoring

* SAAS-175 Extract auth type to const

* SAAS-175 Generate passwords in tests

* SAAS-175 Refresh session every day

* SAAS-175 Save users email

* SAAS-175 Fix permissions

* SAAS-175 Fix tests

* SAAS-175 Refactoring

Co-authored-by: Alexey Palazhchenko <alexey.palazhchenko@percona.com>

* SAAS-175 Add comment

* SAAS-175 Remove redundant error check

* SAAS-175 Improove env variable name

* SAAS-175 Fix comments

* SAAS-175 Refactoring

* SAAS-175 Add LogOut option to settings update

* SAAS-175 Fix linters

* SAAS-175 Install gofumpt

* SAAS-175 Use cleanup

* SAAS-175 Gofumpt

* SAAS-175 Use mock for testing

* SAAS-175 Add timeout to refresh session request

* SAAS-175 Set Platrotm API request timeouts

* SAAS-175 Hit CI

* SAAS-175 Refactoring

* SAAS-175 Refactoring

* SAAS-174 Update pmm dependency

* SAAS-174 Update saas dependency

* SAAS-174 Add password reset method

* SAAS-175 Hit CI

* SAAS-174 Update pmm dependecny

* SAAS-175 SAAS-174 Do not wrap gRPC errors

* SAAS-175 Update saas and pmm dependencies

* SAAS-174 Remove password reset API method

* SAAS-175 Hit CI

* SAAS-175 Update pmm dependency

* SAAS-175 Regenerate mocks

* SAAS-175 Update API branch

Co-authored-by: Alexey Palazhchenko <alexey.palazhchenko@percona.com>
Co-authored-by: Alexey Palazhchenko <alexey.palazhchenko@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants