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

Jump to bottom

[Tor Trac #25140]: Add wildcard support to %include on configuration files #1915

Merged
merged 6 commits into from Aug 12, 2020
Merged

[Tor Trac #25140]: Add wildcard support to %include on configuration files #1915

merged 6 commits into from Aug 12, 2020

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@Jigsaw52 @coveralls @torproject-pusher
@Jigsaw52
Copy link
Contributor

@Jigsaw52 Jigsaw52 commented Jun 3, 2020

No description provided.

@Jigsaw52 Jigsaw52 mentioned this pull request Jun 3, 2020
Closed
@coveralls
Copy link

@coveralls coveralls commented Jun 3, 2020 •
edited

Pull Request Test Coverage Report for Build 9880

  • 132 of 151 (87.42%) changed or added relevant lines in 4 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.03%) to 64.451%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/lib/fs/conffile.c 49 51 96.08%
src/lib/fs/files.c 0 4 0.0%
src/lib/fs/path.c 79 92 85.87%
Files with Coverage Reduction New Missed Lines %
src/lib/fs/conffile.c 1 96.43%
Totals Coverage Status
Change from base Build 9878: 0.03%
Covered Lines: 51623
Relevant Lines: 80097
💛 - Coveralls

@Jigsaw52 Jigsaw52 force-pushed the torrc-include-wildcards-25140_rebasedv2 branch 2 times, most recently from 2ea63c3 to 8ac24ec Jun 3, 2020
@Jigsaw52 Jigsaw52 changed the title [Tor Trac #25140] Add wildcard support to %include on configuration files [Tor Trac #25140]: Add wildcard support to %include on configuration files Jun 6, 2020
Jigsaw52 added 4 commits Jul 15, 2020
@Jigsaw52
Add support for patterns on %include #25140
34fa2c4 
Also adds generic tor_glob function to expand globs.
@Jigsaw52
Add test for torrc %include functionality and seccomp sandbox
49dab42 
Tests %include with files and folders, modifying and reloading
the config file with sandbox enabled and reponse of SAVECONF and
getinfo config-can-saveconf control commmands.
@Jigsaw52
Fix small memory leak in config include tests
3c70f26 
Tests that used options_init_from_string were not freeing calling
config_free_all().
@Jigsaw52
Changes file for #25140
ac2c4b1
@Jigsaw52 Jigsaw52 force-pushed the torrc-include-wildcards-25140_rebasedv2 branch from 8ac24ec to ac2c4b1 Jul 15, 2020
@Jigsaw52
Improved documentation and comments #25140
0b633b1
@Jigsaw52 Jigsaw52 force-pushed the torrc-include-wildcards-25140_rebasedv2 branch from ca552c2 to 60c436d Aug 12, 2020
@Jigsaw52
Attempt to fix test_include.py on TravisCI #25140
60c436d
@torproject-pusher torproject-pusher merged commit e151442 into torproject:master Aug 12, 2020
2 checks passed
@Jigsaw52 Jigsaw52 deleted the torrc-include-wildcards-25140_rebasedv2 branch Aug 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment