nginx-loadbalancer-kubernetes
nginx-loadbalancer-kubernetes copied to clipboard
nginxinc
A Kubernetes Controller to synchronize NGINX+ Resources with Kubernetes Ingress Resources
### Is your feature request related to a problem? Please describe Currently NLK populates the host/port of **all** nodes in the cluster to the NGINX `upstream` block. If a request...
### Is your feature request related to a problem? Please describe [logrus](https://github.com/sirupsen/logrus) is fine, but it's in gone into maintenance mode. Look into the libraries mentioned in the [README](https://github.com/sirupsen/logrus)... At...
### Is your feature request related to a problem? Please describe I'm working on the helm charts for the NLK, found some limitations with the current NLK implementation. ### Describe...
the `go settings.Run()` call is done fairly early in `main()`. The original thought was this would give the Go routine time to load the certificates before the rest of the...
In the Run method, there's a check for nil informer but no such check in initializeEventHandlers method which also accesses the informer. It would be prudent to add a nil...
In handleAddEvent, handleDeleteEvent, and handleUpdateEvent, there's a lack of error handling if the type assertion fails other than logging an error. Consider adding more graceful error handling in these scenarios.
In the NewCertificates method, the Certificates field is initialized to nil, which could cause a panic when trying to insert key-value pairs into the map. It should be initialized to...
NKL is the first thing I think of when I'm trying to remember this project and bring to others attention. Unfortunately, that abbreviation is not in any of the titles...
TLS considerations in client Tracking issue for: - [ ] https://github.com/nginxinc/nginx-k8s-loadbalancer/security/code-scanning/18
### Is your feature request related to a problem? Please describe No problem. Synchronizer should not know about the NGINX Plus client. It should simply provide an HttpClient to the...