trifectatechfoundation / sudo-rs

A memory safe implementation of sudo and su.
Other
2.88k stars 77 forks source link

reject wildcards in argument positions #781

Closed squell closed 4 months ago

squell commented 12 months ago

Closing #780. Also rejects '?'.

I made this PR since there was too much discussion, and that can take place on the linked issue.

codecov[bot] commented 12 months ago

Codecov Report

Patch coverage: 66.66% and no project coverage change.

Comparison is base (6748b1c) 55.33% compared to head (4b953a0) 55.33%. Report is 2 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #781 +/- ## ======================================= Coverage 55.33% 55.33% ======================================= Files 72 72 Lines 9837 9840 +3 ======================================= + Hits 5443 5445 +2 - Misses 4394 4395 +1 ``` | [Files Changed](https://app.codecov.io/gh/memorysafety/sudo-rs/pull/781?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=memorysafety) | Coverage Δ | | |---|---|---| | [src/sudoers/tokens.rs](https://app.codecov.io/gh/memorysafety/sudo-rs/pull/781?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=memorysafety#diff-c3JjL3N1ZG9lcnMvdG9rZW5zLnJz) | `95.90% <66.66%> (-0.53%)` | :arrow_down: |

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