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

Postee as api 2 #163

Closed
wants to merge 79 commits into from
Closed

Postee as api 2 #163

wants to merge 79 commits into from

Conversation

AndreyLevchenko
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Aug 19, 2021

Codecov Report

Merging #163 (2a96139) into main (93c91d2) will decrease coverage by 8.18%.
The diff coverage is 58.44%.

❗ Current head 2a96139 differs from pull request most recent head a788cf7. Consider uploading reports for the commit a788cf7 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main     #163      +/-   ##
==========================================
- Coverage   86.77%   78.59%   -8.19%     
==========================================
  Files          28       19       -9     
  Lines        1104     1093      -11     
==========================================
- Hits          958      859      -99     
- Misses         88      181      +93     
+ Partials       58       53       -5     
Impacted Files Coverage Δ
router/api.go 36.45% <36.45%> (ø)
router/router.go 74.25% <60.54%> (-11.89%) ⬇️
regoservice/eval.go 79.59% <80.00%> (-0.41%) ⬇️
dbservice/dbservice.go 86.66% <86.66%> (ø)
data/utils.go 100.00% <100.00%> (ø)
msgservice/msghandling.go 84.33% <100.00%> (-3.02%) ⬇️
router/anonymizer.go 100.00% <100.00%> (ø)
router/builders.go 97.05% <100.00%> (ø)
router/parsecfg.go 86.66% <100.00%> (ø)
msgservice/scheduler.go 71.42% <0.00%> (-19.05%) ⬇️
... and 9 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 93c91d2...a788cf7. Read the comment docs.

@CLAassistant
Copy link

CLAassistant commented Sep 14, 2021

CLA assistant check
All committers have signed the CLA.

DmitriyLewen and others added 27 commits December 24, 2021 14:52
fix: fixed golangci-lint test errors
fix: fixed golangci-lint test errors
Test: added test for errors to getCfgCacheSource
Reuse postgres db conncetion instance
This commit contains code that moves the rego evaluation code to the
route handling process.
At its current implementation, the output handler checks for rego match - this
check is redundant since the output does not effect the matching process.
@AndreyLevchenko
Copy link
Contributor Author

close in favour of #282

kairi003 pushed a commit to kairi003/postee that referenced this pull request Oct 18, 2022
…curity#163)

Bumps [k8s.io/code-generator](https://github.com/kubernetes/code-generator) from 0.24.1 to 0.24.2.
- [Release notes](https://github.com/kubernetes/code-generator/releases)
- [Commits](kubernetes/code-generator@v0.24.1...v0.24.2)

---
updated-dependencies:
- dependency-name: k8s.io/code-generator
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

5 participants