Roundup Issue Tracker

changeset 29800bbb7c09 branch default bookmark ... [More] tag tip user John Rouillard description feat: issue2551332: roundup-admin updateconfig preserves DEFAULT and interpolation tokens. The main config.ini has supported interpolation with %(name)s tokens in values. However roundup-admin's upgradeconfig would wipe any names that weren't defined in the config schema. Also it would write out the values of the interpolation tokens and not preserve the tokens. This checkin defines a DEFAULT section and preserves the names/values defined in the DEFAULT section of the file. Names/tokens not in the config schema are not preserved in any other section of the file. It also preserves the interpolation tokens. reference.txt: swapped paragraph on schema.py in Tracker Configuration section before the paragraph on config.ini so all config.ini stuff is together. added docs on DEFAULT section admin.py: added better exception handling to prevent full tracebacks on errors generated by interpolation support and DEFAULT section support. put python3 UserDict import in try section not except section. Need to do more investigation before I remove pure UserDict import. in do_genconfig() do early return if we are not updating config. configuration.py: add attribute to store raw version of value for each Option also add method for setting. make format() output in raw mode as well, add rformat() as wrapper. make save() (of config) optionally output in raw mode, write DEFAULT section and default values. use textwrap dedent/wrap for cleaning/reformatting text. remove python 2 support for importing configparser cleanup import order a little. OptionValueError reports file (for detectors/extensions config.ini) _adjust_options: remove unneeded list() around config.defaults().keys() test_admin.py: use pathlib.Path for path munging in new code. fix replace_in_file() to not add an extra newline for every line. add subtests fixture add three tests: config.ini file w/ DEFAULT section with boolean definition that is interpolated. Verify able to display user1. generate updateconfig: verify DEFAULT section, default name/value and interpolation token all exist in new file. change interpolated value to a non boolean value and verify error is reported. files CHANGES.txtdoc/reference.txtroundup/admin.pyroundup/configuration.pytest/test_admin.py [Less]

changeset c6a19f66aff1 branch bookmark ... [More] tag user John Rouillard description test: Make test_instance.py test_registerUtilConflicts work on windows. Use pathlib.Path to generate windows/unix path strings as needed. files test/test_instance.py [Less]

changeset 8d15d37c22ba branch bookmark ... [More] tag user John Rouillard description build(deps): bump github/codeql-action/analyze from 4.37.6 to 4.37.7 #119 files .github/workflows/codeql-analysis.yml [Less]

changeset d71f2dbb9620 branch bookmark ... [More] tag user John Rouillard description build(deps): bump github/codeql-action/autobuild from 4.37.6 to 4.37.7 #118 files .github/workflows/codeql-analysis.yml [Less]

changeset 0b8e386d9a56 branch bookmark ... [More] tag user John Rouillard description build(deps): bump github/codeql-action/upload-sarif from 4.37.6 to 4.37.7 #117 files .github/workflows/anchore.yml.github/workflows/ossf-scorecard.yml [Less]

changeset 3d2e6e3ed607 branch bookmark ... [More] tag user John Rouillard description build(deps): bump github/codeql-action/init from 4.37.6 to 4.37.7 #116 files .github/workflows/codeql-analysis.yml [Less]

changeset b52a972e872f branch bookmark ... [More] tag user mint@localhost description test: adapt test to new error found by hypothesis. Fix the test to turn test cases like: @verbose=&=0 into @verbose= as & is a parameter separator and '=0' is an error. This test case was expecting a 200 return value from the server, but got a 400 as '=0' is missing the parameter name. I was doing the fixup for this case after the test had run, but I need to normalize the string before. Also took two duplicate test cases that ran the same tests against different endpoints and had them call a common base method passing the url in. So I only had to fix the logic ion one place. files test/test_liveserver.py [Less]

changeset 99876ff604f8 branch bookmark ... [More] tag user mint@localhost description issue2551161 - fix ResourceWarnings - unclosed sqlite db connections. Four places in back_anydbm.py could open read only dbm file handles. These handles were local to the function. They were not explicitly closed in 3 cases. I added explicit closing for all 3 places. I reused the existing method of a if db is None: db = something_opening_a db_handle() must_close = True try: do stuff with db finally: if must_close: db.close in once place and just duplicated the return line in the other two sicne it was shorter. Also typed the return value for hasnode() to boolean and made return 1 into return True as we were mixing boolean and integer returns. files CHANGES.txtroundup/backends/back_anydbm.py [Less]

changeset 82d84399b22f branch bookmark ... [More] tag user mint@localhost description issue2551161 - fix ResourceWarnings open log file from configure.py When the .ini or .json log file specs are broken, configure left a partly configured logging setup before it exited. Make sure to logging.shutdown() before exit. files CHANGES.txtroundup/configuration.py [Less]

changeset 783497f0dbc8 branch bookmark ... [More] tag user John Rouillard description fix: roundup-server generate exception source info The final exception catchall didn't decode the traceback and let you know where the error was. It made debugging the issue with registerUtil more difficult than it had to be. So now we get filename, line number, and function/method name. files roundup/scripts/roundup_server.py [Less]