bug: don't reference vendored outside of anypy modules

client.py was referencing the vendored.cgi module. This is wrong.
It should pull the vendored.cgi module only via anypy.cgi_.

Author: rouilj

roundup/cgi/client.py

@@ -59,6 +59,7 @@ class SysCallError(Exception):
 from roundup.logcontext import gen_trace_id, store_trace_reason
 from roundup.mailer import Mailer, MessageSendError
 from roundup.mlink_expr import ExpressionError
+from roundup.performance import report_object_use
 
 logger = logging.getLogger('roundup')
 
@@ -585,6 +586,7 @@ def setTranslator(self, translator=None):
 
     @gen_trace_id()
     @store_trace_reason('client_main')
+    @report_object_use(highest=20, pre_collect=False)
     def main(self):
         """ Wrap the real main in a try/finally so we always close off the db.
         """
@@ -1187,9 +1189,15 @@ def determine_language(self):
         # self.language to the desired language !
         self.language = language
 
-        self.setTranslator(TranslationService.get_translation(
+        try:
+            self.setTranslator(TranslationService.get_translation(
                 language,
                 tracker_home=self.instance.config["TRACKER_HOME"]))
+        except IOError as e:
+            logger.error(str(e), extra={"requested_language": language})
+            # failed to set the requested/TRACKER_LANGUAGE language.
+            # Set to en.
+            self.language = ""
 
     def authenticate_bearer_token(self, challenge):
         ''' authenticate the bearer token. Refactored from determine_user()
@@ -1931,7 +1939,7 @@ def reauth(self, exception):
            Can be overridden by code in tracker's interfaces.py.
         """
 
-        from roundup.anypy.vendored.cgi import MiniFieldStorage
+        from roundup.anypy.cgi_ import MiniFieldStorage
 
         original_action = self.form['@action'].value if '@action' \
             in self.form else ""