Merge pull request #1959 from JdeRobot/1826-manager-add-functionality-to-the-manager-to-store-the-style-evaluation-in-elasticsearch-when-appropriate

Evaluate style on RADI webserver

Author: davidrol6

academy/settings.py

@@ -45,7 +45,8 @@
     'webpack_loader',                                       # Integrate React webpack bundles with Django
     'rest_framework',                                       # Django Rest Framework
     'academy.academy_rest_api.apps.AcademyRestApiConfig',   # Academy rest api application
-    'react_frontend.apps.ReactFrontendConfig'               # React frontend application
+    'react_frontend.apps.ReactFrontendConfig',               # React frontend application
+    'corsheaders',
 ]
 
 MIDDLEWARE = [
@@ -56,6 +57,7 @@
     'django.contrib.auth.middleware.AuthenticationMiddleware',
     'django.contrib.messages.middleware.MessageMiddleware',
     'django.middleware.clickjacking.XFrameOptionsMiddleware',
+    'corsheaders.middleware.CorsMiddleware',
 ]
 
 ROOT_URLCONF = 'academy.urls'
@@ -153,4 +155,29 @@
     'POLL_INTERVAL': 0.1,
     'IGNORE': [r'.+\.hot-update.js', r'.+\.map'],
   }
-}
+}
+
+
+CSRF_TRUSTED_ORIGINS = [
+    "http://localhost:7000",
+    "http://127.0.0.1:7000",
+]
+
+CORS_ALLOWED_ORIGINS = [
+    "http://localhost:7000",
+    "http://127.0.0.1:7000",
+]
+
+CORS_ALLOW_HEADERS = (
+    'accept',
+    'accept-encoding',
+    'authorization',
+    'content-type',
+    'dnt',
+    'origin',
+    'user-agent',
+    'x-csrftoken',
+    'x-requested-with',
+)
+
+CORS_ALLOW_CREDENTIALS = True

exercises/urls.py

@@ -6,4 +6,5 @@
     path('', views.index, name='index'),
     path("exercises/<slug:exercise_id>/", views.load_exercise, name="load_exercise"),
     path("exercise/request/<slug:exercise_id>", views.request_code, name="request_code"),
+    path('evaluate_style/', views.evaluate_style, name="evaluate_style"),
 ]

exercises/views.py

@@ -3,7 +3,11 @@
 from django.conf import settings
 from .models import Exercise
 import ast
+import sys
 import json
+import tempfile
+from pylint import epylint as lint
+from django.views.decorators.csrf import csrf_exempt
 
 #TODO: Too many hardcoded strings, review
 def index(request):
@@ -30,3 +34,50 @@ def request_code(request, exercise_id):
     if difficulty != None:
         print('EXERCISE: ', exercise_id, 'DIFFICULTY: ', difficulty)
         return HttpResponse(data, content_type="text/plain")
+
+@csrf_exempt
+def evaluate_style(request):
+    print("1º")
+    try:
+        python_code = get_python_code(request)
+        code_file = tempfile.NamedTemporaryFile(delete=False)
+        code_file.write(python_code.encode())
+        code_file.seek(0)
+        options = code_file.name + ' --enable=similarities' + " --disable=C0114,C0116"
+        (stdout, stderr) = lint.py_run(options, return_std=True)
+        code_file.seek(0)
+        code_file.close()
+        result = stdout.getvalue()
+        name = code_file.name.split('/')[-1]
+        result = result[(result.find(name) + len(name) - 1):]
+        result = result.replace(code_file.name, 'mycode')
+        result = result[result.find('\n'):]
+        init_index = result.find('rated at ')
+        score = -1
+        if init_index != -1:
+            init_index += len('rated at ')
+            final_index = result.find('/10', init_index)
+            score = round(float(result[init_index:final_index]) * 10, 2)
+        response = HttpResponse(result+"\n"+str(score), content_type="text/plain")
+        return response
+    except Exception as ex:
+        print("2º")
+        print(ex)
+        response = HttpResponse("Error", content_type="text/plain")
+        return response
+
+
+
+
+def get_python_code(request):
+    python_code = request.GET.get('python_code', None)
+    if not python_code:
+        body_unicode = request.body.decode('utf-8')
+        body_unicode = body_unicode[0:18] + body_unicode[18: len(body_unicode) - 2].replace('"',
+                                                                                            "'") + body_unicode[-2:]
+        body = json.loads(body_unicode, strict=False)
+        python_code = body['python_code']
+    python_code = python_code.lstrip('\\').lstrip('"')
+    python_code = python_code.replace('\\n', '\n')
+    python_code = python_code.replace('\\"', '"').replace("\\'", "'")
+    return python_code

scripts/Dockerfile.base

@@ -267,7 +267,7 @@ RUN apt-get update && apt-get -y --quiet --no-install-recommends install \
 	&& rm -rf /var/lib/apt/lists/{apt,dpkg,cache,log} /tmp/* /var/tmp/*
 
 # websocket server dependency
-RUN pip3 install websocket_server posix-ipc django djangorestframework==3.13.1 django-webpack-loader==1.5.0
+RUN pip3 install websocket_server posix-ipc django djangorestframework==3.13.1 django-webpack-loader==1.5.0 django-cors-headers==3.14.0
 RUN python3.8 -m pip install websockets asyncio
 
 # RAM dependencies

scripts/requirements.txt

@@ -29,6 +29,7 @@ dill==0.3.5.1
 distro==1.4.0
 distro-info===0.23ubuntu1
 Django==4.1.7
+django-cors-headers==3.14.0
 django-webpack-loader==1.5.0
 djangorestframework==3.13.1
 docutils==0.16