biscuit-auth / biscuit-rust

Rust implementation of the Biscuit authorization token
https://www.biscuitsec.org
209 stars 29 forks source link

add new Array and Map types #219

Open Geal opened 4 months ago

Geal commented 4 months ago

this tries to enforce that all array elements are of the same type, but this is not very strict at the moment, it does not look at lower levels of composite types.

breaking changes:

Todo:

codspeed-hq[bot] commented 4 months ago

CodSpeed Performance Report

Merging #219 will not alter performance

Comparing map-array (ab861d1) with v5 (22bd667)

Summary

✅ 12 untouched benchmarks

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 72.92419% with 75 lines in your changes missing coverage. Please review.

Project coverage is 68.83%. Comparing base (525c278) to head (0399b2f).

:exclamation: Current head 0399b2f differs from pull request most recent head d5003f3

Please upload reports for the commit d5003f3 to get more accurate results.

Files Patch % Lines
biscuit-parser/src/builder.rs 41.46% 24 Missing :warning:
biscuit-auth/src/token/builder.rs 78.88% 19 Missing :warning:
biscuit-auth/src/datalog/expression.rs 81.33% 14 Missing :warning:
biscuit-parser/src/parser.rs 55.00% 9 Missing :warning:
biscuit-auth/src/datalog/mod.rs 0.00% 4 Missing :warning:
biscuit-auth/src/format/convert.rs 78.94% 4 Missing :warning:
biscuit-auth/src/datalog/symbol.rs 93.33% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## v5 #219 +/- ## ========================================== + Coverage 68.57% 68.83% +0.25% ========================================== Files 25 25 Lines 5681 5932 +251 ========================================== + Hits 3896 4083 +187 - Misses 1785 1849 +64 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.