From 703db441bf52cbf4a17eb729ead8f037d8dccd59 Mon Sep 17 00:00:00 2001
From: Adar Nimrod <nimrod@shore.co.il>
Date: Sat, 16 Jul 2022 17:28:18 +0300
Subject: [PATCH] Some pre-commit fixes.

---
 Dockerfile | 2 ++
 README.md  | 2 +-
 app.py     | 2 ++
 3 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/Dockerfile b/Dockerfile
index 5eb5e2e..bfb1c0e 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -9,6 +9,7 @@ RUN apt-get update && \
     ;
 WORKDIR /wheels
 RUN python3 -m pip wheel https://github.com/python-ldap/python-ldap/releases/download/python-ldap-3.3.1/python-ldap-3.3.1.tar.gz
+# hadolint ignore=DL3059
 RUN python3 -m pip wheel git+https://github.com/adarnimrod/flask-simpleldap.git@ldapi-support#egg=flask-simpleldap
 
 FROM docker.io/library/python:3.10-slim-bullseye
@@ -21,6 +22,7 @@ RUN apt-get update && \
     && \
     rm -rf /tmp/* /var/tmp/* /var/lib/apt/lists/* /var/cache/apt/archives/*
 COPY --from=wheels /wheels/*.whl /wheels/
+# hadolint ignore=DL3042
 RUN pip install /wheels/*.whl
 WORKDIR /app
 COPY requirements.txt ./
diff --git a/README.md b/README.md
index 4361536..32044ca 100644
--- a/README.md
+++ b/README.md
@@ -2,7 +2,7 @@
 
 [![pipeline status](https://git.shore.co.il/shore/ldap-auth/badges/master/pipeline.svg)](https://git.shore.co.il/shore/ldap-auth/-/commits/master)
 
-LDAP authentication webserver to use with Nginx' auth\_request. See this [blog
+LDAP authentication webserver to use with Nginx' auth_request. See this [blog
 post](https://www.shore.co.il/blog/ldap-auth/) for further explanation.
 
 ## Configuration
diff --git a/app.py b/app.py
index 965e807..5f93c0c 100644
--- a/app.py
+++ b/app.py
@@ -2,6 +2,7 @@
 # pylint: disable=import-error
 
 import os
+
 from flask import Flask
 from flask_simpleldap import LDAP
 
@@ -44,6 +45,7 @@ def ping():
 @app.route("/validate")
 @ldap.basic_auth_required
 def login():
+    """Login to the app, requires LDAP authentication."""
     return "OK"
 
 
-- 
GitLab