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

refact: invalidate auth client cache & merge auth filter log #1528

Merged
merged 2 commits into from
Jun 30, 2021

Conversation

imbajin
Copy link
Member

@imbajin imbajin commented Jun 29, 2021

  1. merge auth filter log into audit.log
  2. invalidate auth client role cache with CUD operation

1. merge auth filter log into audit.log
2. invalidate auth client role cache with CUD operation
3.
@codecov
Copy link

codecov bot commented Jun 29, 2021

Codecov Report

Merging #1528 (a7c7485) into master (f41c6e9) will decrease coverage by 0.43%.
The diff coverage is 43.33%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1528      +/-   ##
============================================
- Coverage     59.67%   59.23%   -0.44%     
+ Complexity     6151     6131      -20     
============================================
  Files           410      410              
  Lines         33260    33272      +12     
  Branches       4591     4590       -1     
============================================
- Hits          19847    19710     -137     
- Misses        11340    11495     +155     
+ Partials       2073     2067       -6     
Impacted Files Coverage Δ
...a/com/baidu/hugegraph/auth/HugeGraphAuthProxy.java 57.34% <21.05%> (-0.63%) ⬇️
.../java/com/baidu/hugegraph/auth/RolePermission.java 91.76% <81.81%> (ø)
...graph/backend/store/scylladb/ScyllaDBFeatures.java 0.00% <0.00%> (-100.00%) ⬇️
...h/backend/store/scylladb/ScyllaDBTablesWithMV.java 0.00% <0.00%> (-96.04%) ⬇️
.../backend/store/scylladb/ScyllaDBStoreProvider.java 0.00% <0.00%> (-94.60%) ⬇️
...egraph/backend/store/cassandra/CassandraShard.java 50.92% <0.00%> (-1.86%) ⬇️
.../backend/store/cassandra/CassandraSessionPool.java 56.12% <0.00%> (-1.03%) ⬇️
...a/com/baidu/hugegraph/backend/query/Condition.java 78.10% <0.00%> (-0.73%) ⬇️
.../baidu/hugegraph/backend/query/ConditionQuery.java 86.00% <0.00%> (-0.41%) ⬇️
...c/main/java/com/baidu/hugegraph/task/HugeTask.java 71.69% <0.00%> (-0.31%) ⬇️
... and 3 more

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 f41c6e9...a7c7485. Read the comment docs.

@zhoney zhoney merged commit 23beacd into master Jun 30, 2021
@zhoney zhoney deleted the fix-invalidate-cache branch June 30, 2021 08:38
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.

None yet

4 participants