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

Add permission and new API in log-manager #5046

Merged
merged 23 commits into from
Nov 11, 2020
Merged

Add permission and new API in log-manager #5046

merged 23 commits into from
Nov 11, 2020

Conversation

Binyang2014
Copy link
Contributor

  • Add permission check in log-manage
  • Add log-manager API to retrieve log

@Binyang2014 Binyang2014 marked this pull request as ready for review November 3, 2020 05:49
@Binyang2014 Binyang2014 mentioned this pull request Nov 3, 2020
Closed
8 tasks
@coveralls
Copy link

coveralls commented Nov 3, 2020 •
edited
Loading

Coverage Status

Coverage decreased (-0.2%) to 34.223% when pulling b26c2d6 on binyli/log-manager into d434a3c on master.

@Binyang2014 Binyang2014 merged commit 2ccdc1b into master Nov 11, 2020
@Binyang2014 Binyang2014 deleted the binyli/log-manager branch November 11, 2020 10:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@hzy46 hzy46 hzy46 approved these changes

@abuccts abuccts Awaiting requested review from abuccts

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Binyang2014 @coveralls @hzy46