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.

Sign up for GitHub

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

fix nil pointer #119

Merged
merged 8 commits into from
Apr 18, 2023
Merged

fix nil pointer #119

merged 8 commits into from
Apr 18, 2023

Conversation

till
Copy link
Contributor

@till till commented May 14, 2022

  • Refactor: move all dns utiliy functions
  • Fix: potential fix for nil pointer

@till till self-assigned this May 14, 2022
@till till mentioned this pull request May 14, 2022
Closed
@till till force-pushed the fix-nil-pointer branch 2 times, most recently from 167e4be to 364d42e Compare May 16, 2022 10:27
@till till mentioned this pull request Sep 6, 2022
Merged
till added 8 commits September 6, 2022 21:23
@till
Refactor: move all dns utiliy functions
99fbd8a 
Creates a package which wraps miekg/dns calls.

Hopefully makes reading and testing this code easier in the future.
@till
Fix: potential fix for nil pointer
23885ae 
When no IPs are returned, we should stop this dance. This could be
a temporary problem with dns resolution.

For: #108
@till
Fix: run tests locally
ab4ebde
@till
Chore: refactor rbl code into a pkg
b4039bc
@till
Chore: build containers
7b9195b 
- os: linux
- arch: amd64, arm64

Resolves: #84
@till
Chore: switch to debian (slim)
2c3c4e4
@till
Chore: test release build
11d4a0d
@till
Fix: release process (container)
5244f1a
@till
Copy link
Contributor Author

till commented Apr 18, 2023

Merging this finally. Will fix the release test separately.

@till till merged commit cc8a46b into main Apr 18, 2023
@till till deleted the fix-nil-pointer branch April 18, 2023 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@till till

Labels
size/L
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@till