Skip to content
This repository

quick hack to deal with issue of converting the string '0A1' to an int #17

wants to merge 1 commit into from

2 participants

M. David Peterson Stefan de Konink
M. David Peterson

This issue only makes itself known when running cherokee-admin with -x and otherwise causes cherokee-admin to be inaccessible with a 503 timeout error.

Stefan de Konink

I think we should deal with this in a better way. Either a regular expression to prevent future problems, because it already splits on 'b'.

Stefan de Konink skinkie closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Jan 10, 2012
M. David Peterson quick hack to deal issue of converting the string '0A1' to an int (wh…
…ich makes cherokee-admin inaccessible)
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 1 addition and 1 deletion. Show diff stats Hide diff stats

  1. 2  admin/
2  admin/
@@ -88,7 +88,7 @@ def config_version_get_current():
88 88
89 89
     major = int(v1)
90 90
     minor = int(v2)
-    micro = int(v3)
+    micro = int(v3.strip('A1'))    
92 92
93 93
     return "%03d%03d%03d" %(major, minor, micro)
94 94

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.