Skip to content

Conversation

@tofu-rocketry
Copy link
Member

No description provided.

tofu-rocketry and others added 19 commits December 16, 2020 17:40
They use dashes rather than dots.
Rather than fetching the host certificate config twice, we can reuse the
varible that it gets read into.
Adds server and host dns to the sender log
Globals are generally not a good idea so replace the use of a global
logger with passing the logger to the function that requires it.
- Fix the function so that it supports both receivers and senders.
- Add checking of the protocol set so that invalid protocols lead to a
sys.exit(1).
- Set the logging level to WARN if the default protocol is set as it
should be brought to the user's attention that something isn't quite
right in their config.
@tofu-rocketry tofu-rocketry added this to the 3.2.0 milestone Mar 18, 2021
@tofu-rocketry tofu-rocketry self-assigned this Mar 18, 2021
@tofu-rocketry tofu-rocketry requested a review from a team as a code owner March 18, 2021 15:49
Copy link
Member

@gregcorbett gregcorbett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume this is all previously reviewed code.

@tofu-rocketry tofu-rocketry merged commit bb2f026 into apel:master Mar 18, 2021
@tofu-rocketry tofu-rocketry deleted the release-3.2.0 branch March 18, 2021 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants