Show endpoint URL instead of server address input for WebDAV

since the whole URL is required, not only the IP or hostname. Merge with S3 endpoint URL setting, but show a different tooltop.
motioneye-project · May 1, 2022 · 66cd69e · 66cd69e
1 parent 43cd07d
commit 66cd69e
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 15 deletions.
diff --git a/motioneye/templates/main.html b/motioneye/templates/main.html
@@ -459,7 +459,7 @@
                             </td>
                             <td><span class="help-mark" title="{{ _("elektu servon, al kiu la mediadosieroj estu alŝutitaj") }}">?</span></td>
                         </tr>
-                        <tr class="settings-item" required="true" depends="uploadEnabled uploadService=(ftp|sftp|http|https|webdav)">
+                        <tr class="settings-item" required="true" depends="uploadEnabled uploadService=(ftp|sftp|http|https)">
                             <td class="settings-item-label"><span class="settings-item-label">{{ _("Servila Adreso") }}</span></td>
                             <td class="settings-item-value"><input type="text" class="styled storage camera-config" id="uploadServerEntry"></td>
                             <td><span class="help-mark" title="{{ _("la domajna nomo aŭ IP-adreso de la servilo (ekz. ftp.example.com aŭ 192.168.1.3)") }}">?</span></td>
@@ -479,6 +479,14 @@
                             </td>
                             <td><span class="help-mark" title="{{ _("la HTTP-metodo uzi por alŝuti dosierojn") }}">?</span></td>
                         </tr>
+                        <tr class="settings-item" depends="uploadEnabled uploadService=(s3|webdav)">
+                            <td class="settings-item-label"><span class="settings-item-label">{{ _("Endpoint URL") }}</span></td>
+                            <td class="settings-item-value"><input type="text" class="styled storage camera-config" id="uploadEndpointUrlEntry"></td>
+                            <td>
+                                <span depends="uploadService=(s3)" class="help-mark" title="{{ _("The complete URL to the S3 server endpoint, e.g. http://my.minio:9000. Leave this empty for AWS!") }}">?</span>
+                                <span depends="uploadService=(webdav)" class="help-mark" title="{{ _("The complete URL to the WebDAV server endpoint, e.g. https://my.cloud/remote.php/dav/files/Me/") }}">?</span>
+                            </td>
+                        </tr>
                         <tr class="settings-item" required="true" depends="uploadEnabled uploadService!=(s3)">
                             <td class="settings-item-label"><span class="settings-item-label">{{ _("Loko") }}</span></td>
                             <td class="settings-item-value"><input type="text" class="styled storage camera-config" id="uploadLocationEntry"></td>
@@ -518,11 +526,6 @@
                             </td>
                             <td><span class="help-mark" title="{{ _("alklaku ĉi tie por akiri la ŝlosilan rajtigan servon") }}">?</span></td>
                         </tr>
-                        <tr class="settings-item" depends="uploadEnabled uploadService=(s3)">
-                            <td class="settings-item-label"><span class="settings-item-label">{{ _("Endpoint URL") }}</span></td>
-                            <td class="settings-item-value"><input type="text" class="styled storage camera-config" id="uploadEndpointUrlEntry"></td>
-                            <td><span class="help-mark" title="{{ _("The complete URL to the S3 server endpoint, e.g. http://my.minio:9000. Leave this empty for AWS!") }}">?</span></td>
-                        </tr>
                         <tr class="settings-item" depends="uploadEnabled uploadService=(s3)">
                             <td class="settings-item-label"><span class="settings-item-label">{{ _("Access Key") }}</span></td>
                             <td class="settings-item-value"><input type="password" autocomplete="new-password" class="styled storage camera-config" id="uploadAccessKeyEntry"></td>
@@ -1108,8 +1111,8 @@
                             <td><span class="help-mark" title="{{ _("difinas la bildan serĉtempan intervalon por krei telegramajn aldonojn (pli altaj valoroj generas pli da bildoj koste de pliigita sciiga prokrasto); agordi al 0 por malebligi bildajn aldonaĵojn; vi devas ankaŭ ebligi Senmovajn Bildojn por ke ĉi tio funkciu; vi volos ludi per ĉi tiu numero ĝis bildo estos sendita. Bona komenca numero estas 30 se vi agordas senmovajn bildojn al unu bildmoviĝo ekigita. Por norma movado ekigita, starigu ĉi tion multe pli malalte.") }}">?</span></td>
                         </tr>
                         <tr class="settings-item" required="true" depends="telegramNotificationsEnabled motionDetectionEnabled" strip="true">
-				<td class="settings-item-label"><a href="https://core.telegram.org/bots#6-botfather" target="_blank"><div class="button normal-button test-button" id="apiInstructionButton" style="opacity: 1; display: inline-block;">{{ _("API-Informoj") }}</div></a></td>
-				<td class="settings-item-label"><div class="button normal-button test-button" id="telegramTestButton" style="opacity: 1; display: inline-block;">{{ _("Testo") }}</div></td>
+                            <td class="settings-item-label"><a href="https://core.telegram.org/bots#6-botfather" target="_blank"><div class="button normal-button test-button" id="apiInstructionButton" style="opacity: 1; display: inline-block;">{{ _("API-Informoj") }}</div></a></td>
+                            <td class="settings-item-label"><div class="button normal-button test-button" id="telegramTestButton" style="opacity: 1; display: inline-block;">{{ _("Testo") }}</div></td>
                         </tr>
                         <tr class="settings-item" depends="motionDetectionEnabled">
                             <td colspan="100"><div class="settings-item-separator"></div></td>

diff --git a/motioneye/uploadservices.py b/motioneye/uploadservices.py
@@ -886,7 +886,7 @@ class Webdav(UploadService):
     NAME = 'webdav'
 
     def __init__(self, camera_id):
-        self._server = None
+        self._endpoint_url = None
         self._username = None
         self._password = None
         self._location = None
@@ -914,7 +914,7 @@ def _request(self, url, method, body=None):
                 raise e
 
     def _make_dirs(self, path):
-        dir_url = self._server.rstrip('/') + '/'
+        dir_url = self._endpoint_url.rstrip('/') + '/'
         for folder in path.split('/'):
             dir_url = dir_url + folder + '/'
             self._request(dir_url, 'MKCOL')
@@ -923,7 +923,7 @@ def test_access(self):
         try:
             path = self._location.strip('/') + '/' + str(time.time())
             self._make_dirs(path)
-            self._request(self._server.rstrip('/') + '/' + path, 'DELETE')
+            self._request(self._endpoint_url.rstrip('/') + '/' + path, 'DELETE')
             return True
         except Exception as e:
             self.error(str(e), exc_info=True)
@@ -935,23 +935,23 @@ def upload_data(self, filename, mime_type, data, ctime, camera_name):
         self._make_dirs(path)
         self.debug(f'uploading {filename} of {len(data)} bytes')
         self._request(
-            self._server.rstrip('/') + '/' + path + '/' + filename,
+            self._endpoint_url.rstrip('/') + '/' + path + '/' + filename,
             'PUT',
             bytearray(data),
         )
         self.debug('upload done')
 
     def dump(self):
         return {
-            'server': self._server,
+            'endpoint_url': self._endpoint_url,
             'username': self._username,
             'password': self._password,
             'location': self._location,
         }
 
     def load(self, data):
-        if data.get('server') is not None:
-            self._server = data['server']
+        if data.get('endpoint_url') is not None:
+            self._endpoint_url = data['endpoint_url']
         if data.get('username') is not None:
             self._username = data['username']
         if data.get('password') is not None: