stakater / Forecastle

Forecastle is a control panel which dynamically discovers and provides a launchpad to access applications deployed on Kubernetes – [✩Star] if you're using it!
https://stakater.com
Apache License 2.0
588 stars 59 forks source link

[BUG] Forecastle fails on empty TLS field in ingress #407

Closed MuneebAijaz closed 1 month ago

MuneebAijaz commented 10 months ago

Describe the bug It seems like forecastle panics on empty tls field in ingress

To Reproduce Create an Ingress with

  tls:
  - {} 

field, link it to a ForecastleApp CR. Forecastle pods will show panics. 2023/11/22 15:44:09 http: panic serving [::1]:38128: runtime error: index out of range [0] with length 0 Expected behavior There should be no panics, instead a nil/empty check

Screenshots If applicable, add screenshots to help explain your problem.

Environment

Additional context Add any other context about the problem here.

MuneebAijaz commented 1 month ago

wasnt able to reproduce