Skip to content

Commit

Permalink
Merge pull request #731 from dolfinus/ingress_escape_none
Browse files Browse the repository at this point in the history
[KubeIngressProxy] Do not try to escape None
  • Loading branch information
consideRatio committed May 17, 2023
2 parents 5c6801b + bdc24bf commit 0c837e2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions kubespawner/proxy.py
Original file line number Diff line number Diff line change
Expand Up @@ -349,12 +349,12 @@ def _expand_user_properties(self, template, routespec, data):
if hub_namespace == "default":
hub_namespace = "user"

raw_username = data.get('user')
raw_username = data.get('user') or ''
safe_username = escapism.escape(
raw_username, safe=safe_chars, escape_char='-'
).lower()

raw_servicename = data.get('services')
raw_servicename = data.get('services') or ''
safe_servicename = escapism.escape(
raw_servicename, safe=safe_chars, escape_char='-'
).lower()
Expand Down

0 comments on commit 0c837e2

Please sign in to comment.