F14 : When editing the obs config file, the website return yaml syntax errors

Alexis Koralewski
1 parent a444ac84
Showing 2 changed files with 30 additions and 8 deletions Show diff stats
src/core/pyros_django/obsconfig/templates/obsconfig/edit_config.html
src/core/pyros_django/obsconfig/views.py
@@ -26,6 +26,11 @@
         left:80vw;
         top: 30vh;
     }
+    #message{
+        position:absolute;
+        left:80vw;
+        top: 25vh;
+    }
     .red{
         color:red;
@@ -46,8 +51,8 @@
 <button class="btn-primary" id="check_config">Check schema</button>
 </div>
-<div id="save" class="hidden">
 <p id="message"></p>
+<div id="save" class="hidden">
     {% comment %}
 <label> File name: (without ".yml" extension)</label>
@@ -74,7 +79,6 @@
             editor.resize();
         });
     document.getElementById("editor").addEventListener("input",function(){
-            console.log("input")
             $("#save").attr("class",'hidden')
             $("#save_btn").removeAttr("disabled");
             $("#save_message").empty();
@@ -84,7 +88,6 @@
         csrf_value = $("[name='csrfmiddlewaretoken']").val();
         $.post('{% url 'verify_config' %}', { "config": editor.getValue(), csrfmiddlewaretoken: csrf_value }, function(data,status) {
             console.log(data);
-            console.log(status);
             if(data["is_valid"] == true){
                 $("#message").empty().attr('class', 'normal').append("Valid configuration, do you want to save it ?");
                 $("#save").toggleClass('hidden');
@@ -93,12 +96,20 @@
             else{
                 $("#save").attr("class",'hidden')
                 $("#message").empty().attr('class', 'red').append("Invalid configruation, see the errors below:");
-                for(error of data["message"] ){
-                    for(message of error){
-                        $("#message").append("<p>"+message+"</p>");
+                if (data["message"]){
+                    for(error of data["message"] ){
+                        for(message of error){
+                            $("#message").append("<p>"+message+"</p>");
-                    }
+                        }
+                    }
+                }
+                console.log(data["yaml_error_message"])
+                if (data["yaml_error_message"]){
+                    console.log("yaml_error_message")
+                    let message = data["yaml_error_message"];
+                    $("#message").append("<p><pre>"+message+"</pre></p>");
                 }
             }
         });
@@ -243,7 +243,18 @@ def verify_config(request):
         temp_config_file.write(request.POST.get("config"))
         temp_config_file.seek(0)
         response_data = {}
-        config_file = yaml.safe_load(temp_config_file.read())
+        try:
+            config_file = yaml.safe_load(temp_config_file.read())
+        except yaml.YAMLError as exc:
+            if hasattr(exc, 'problem_mark'):
+                yaml_error_message = ""
+                if exc.context != None:
+                    yaml_error_message += str(exc.problem_mark) + '\n  ' + str(exc.problem) + ' ' + str(exc.context) 
+                else:
+                    yaml_error_message = str(exc.problem_mark.name) + '\n  ' +str(exc.problem)
+                response_data["is_valid"] = False
+                response_data["yaml_error_message"] = yaml_error_message
+            return HttpResponse(json.dumps(response_data), content_type="application/json")
         temp_config_file.seek(0)
         schema = config_file["schema"]
         errors = []