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

Adding TLS to node exporter. Server Auth only. #1198

Closed
wants to merge 54 commits into from
Closed
Changes from 1 commit
Commits
Show all changes
54 commits
Select commit Hold shift + click to select a range
b188e1a
Added TLS listener to node exporter. Server auth only. Pass in cert a…
achiuBAE Dec 12, 2018
7bc2aae
html head tag indent corrected back to normal. no code change
achiuBAE Dec 12, 2018
288bec4
Certificate rotation added requires testing
ksherryBAE Jan 11, 2019
47c0c3b
Certificates reloaded on intial ClientHello
ksherryBAE Jan 31, 2019
65a3a7f
Fixed Spelling
ksherryBAE Jan 31, 2019
fe02aa3
Merge branch 'master' of https://github.com/prometheus/node_exporter …
ksherryBAE Jan 31, 2019
7cdd689
minor formatting
ksherryBAE Jan 31, 2019
379cc25
https package created code duplicated and utilised cleanup required
ksherryBAE Jan 31, 2019
4e1d10a
Spelling fix
ksherryBAE Feb 1, 2019
9df2228
Merge branch 'master' of https://github.com/prometheus/node_exporter …
ksherryBAE Feb 6, 2019
4a8ed90
Minor code formatting
ksherryBAE Feb 6, 2019
3785913
Minor code formatting for CI
ksherryBAE Feb 6, 2019
d5a9dba
Minor code formatting for CI - imports properly ordered
ksherryBAE Feb 6, 2019
96c912b
Struct restructuring
ksherryBAE Feb 6, 2019
a878951
Merge branch 'tls-server-auth' into tls-server-auth-package
ksherryBAE Feb 7, 2019
e62d1a5
Struct adjustments
ksherryBAE Feb 7, 2019
d91ddcb
Merge branch 'tls-server-auth-package' of https://github.com/BAESyste…
ksherryBAE Feb 7, 2019
f232e11
Config extracted out to https package
ksherryBAE Feb 7, 2019
57ff43f
Unecessary struct removed
ksherryBAE Feb 7, 2019
1e2777c
Files cleaned up extraction of server creation started
ksherryBAE Feb 15, 2019
9be5023
Reset tlsConfig to defaults
ksherryBAE Feb 15, 2019
8bf8714
Reset tlsConfig to defaults code formatted
ksherryBAE Feb 15, 2019
0552afe
Packages added
ksherryBAE Feb 21, 2019
b844599
removed go. files
ksherryBAE Feb 21, 2019
5fb1cda
moved GetCertificate into GetTLSConfig
jritchieBAE Feb 21, 2019
9c3e40a
Yaml struct importing
ksherryBAE Feb 25, 2019
9b533b2
-mod merged in
ksherryBAE Feb 25, 2019
186ca13
Functional Yaml Reading
ksherryBAE Feb 25, 2019
0e5792c
Config yaml without CA's added
ksherryBAE Feb 25, 2019
c28ef3c
Cert and Key moved from flags to config
ksherryBAE Feb 26, 2019
55f9b51
CA pools added to config
ksherryBAE Feb 26, 2019
eafb071
Removed conntrack from vendor
ksherryBAE Feb 26, 2019
e7e360b
tls flag renamed tls-config
ksherryBAE Feb 26, 2019
ed87bca
minor formatting
ksherryBAE Feb 26, 2019
55c2e45
Remove local file path references
ksherryBAE Feb 26, 2019
51cbd1e
updated git ignore to ignore yaml and yml extensions
ksherryBAE Feb 26, 2019
632fca9
renamed config base
ksherryBAE Feb 26, 2019
440d75f
Re-added yaml&yml to gitignore
ksherryBAE Feb 26, 2019
cf6a026
tls.yml removed from /node_exporter
ksherryBAE Feb 26, 2019
7d05f7e
ran gofmt
jritchieBAE Feb 27, 2019
b49f96e
log.Fatalf now log.Fatal
jritchieBAE Feb 27, 2019
2fc0145
renamed so not exported from module
jritchieBAE Feb 27, 2019
4bb0273
BuildNameToCertificate moved to certificate init
jritchieBAE Feb 27, 2019
17b13f4
moved declaration of tls.Config{} into loadConfigFromYaml
jritchieBAE Feb 27, 2019
186e533
server declaration to align listen calls
jritchieBAE Feb 27, 2019
1f28e2a
separate listen calls combined into convenience method https.Listen(*…
jritchieBAE Feb 27, 2019
95c3aa3
dependencies added to go.sum
jritchieBAE Feb 27, 2019
4109175
renamed struct so it can be exported
jritchieBAE Feb 27, 2019
f3f5d3f
split function into two to allow code to be reused
jritchieBAE Feb 27, 2019
8b34555
TLS extracted to https package with config file for options
ksherryBAE Feb 27, 2019
baa0545
Spelling and removal of redundant bool comparisons, tidy up of spelli…
ksherryBAE Feb 27, 2019
ed5005f
Merge remote-tracking branch 'prometheus/master' into tls-server-auth
ksherryBAE Mar 6, 2019
45812d4
Update README.md
ksherryBAE Mar 7, 2019
959b61f
Update README.md
ksherryBAE Mar 7, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update README.md
ksherryBAE authored Mar 7, 2019
commit 45812d4850e8c27358ae08e4e163cdde4512b663
3 changes: 2 additions & 1 deletion https/README.md
Original file line number Diff line number Diff line change
@@ -7,7 +7,7 @@ If the config is kept within the https directory

The layout of the config file should be as below:

#TLS CONFIG YAML
`#TLS CONFIG YAML
# Paths to Cert File & Key file from base directory
# Both required for valid tls
# Paths set as string values
@@ -41,3 +41,4 @@ tlsConfig :
# Controls whether a client verifies the servers cert chain and hostname
# Boolean value - TLS insecure if true so should only be set as true for testing
insecureSkipVerify : ~
`