Added flask support #70 #75 #71

Merged
edraft merged 107 commits from #70 into 0.3 2022-11-05 13:55:42 +01:00
Owner
#70 #72 #75 #83 #85 #86
edraft added 1 commit 2022-10-13 19:58:13 +02:00
edraft changed title from Added flask support #70 to WIP: Added flask support #70 2022-10-13 19:58:15 +02:00
edraft added this to the Sprint 0.3 milestone 2022-10-13 19:58:23 +02:00
edraft added this to the Sprint 0.3 project 2022-10-13 19:58:25 +02:00
edraft added 1 commit 2022-10-13 20:57:50 +02:00
edraft added 1 commit 2022-10-14 07:04:59 +02:00
edraft added 1 commit 2022-10-14 07:16:15 +02:00
edraft added 1 commit 2022-10-14 07:51:32 +02:00
edraft added 1 commit 2022-10-14 10:36:02 +02:00
edraft added 1 commit 2022-10-14 13:05:56 +02:00
edraft added 1 commit 2022-10-14 13:17:28 +02:00
edraft added 1 commit 2022-10-14 15:37:55 +02:00
edraft added 2 commits 2022-10-15 00:43:01 +02:00
edraft added 1 commit 2022-10-15 03:04:21 +02:00
edraft added 2 commits 2022-10-15 12:54:20 +02:00
edraft added 3 commits 2022-10-15 17:30:43 +02:00
edraft added 1 commit 2022-10-15 22:52:58 +02:00
edraft force-pushed #70 from 98c1f40024 to 029b46d7de 2022-10-15 22:57:20 +02:00 Compare
edraft added 4 commits 2022-10-16 00:13:20 +02:00
edraft force-pushed #70 from e564956813 to 651482a1b9 2022-10-16 01:55:55 +02:00 Compare
edraft added 1 commit 2022-10-16 10:11:27 +02:00
edraft added 3 commits 2022-10-16 13:09:40 +02:00
edraft added 1 commit 2022-10-16 15:39:35 +02:00
edraft added 1 commit 2022-10-16 16:02:56 +02:00
edraft added 1 commit 2022-10-16 21:10:23 +02:00
edraft added 1 commit 2022-10-16 21:16:08 +02:00
edraft added 1 commit 2022-10-16 21:30:01 +02:00
edraft added 1 commit 2022-10-16 21:36:11 +02:00
edraft changed title from WIP: Added flask support #70 to Added flask support #70 2022-10-17 10:38:39 +02:00
edraft changed title from Added flask support #70 to WIP: Added flask support #70 2022-10-17 10:39:04 +02:00
edraft added 12 commits 2022-10-18 16:37:05 +02:00
edraft added 1 commit 2022-10-18 18:04:55 +02:00
edraft added 1 commit 2022-10-18 18:33:07 +02:00
edraft added 1 commit 2022-10-18 19:50:16 +02:00
edraft added 1 commit 2022-10-18 21:00:26 +02:00
edraft added 1 commit 2022-10-19 16:57:07 +02:00
edraft added 1 commit 2022-10-19 18:01:14 +02:00
edraft added 1 commit 2022-10-19 18:42:23 +02:00
edraft changed title from WIP: Added flask support #70 to Added flask support #70 2022-10-19 18:56:22 +02:00
edraft requested review from Ebola-Chan 2022-10-19 18:56:29 +02:00
edraft requested review from edraft-dev 2022-10-19 18:56:29 +02:00
edraft added the
refactoring
label 2022-10-19 18:56:36 +02:00
edraft self-assigned this 2022-10-19 18:56:39 +02:00
edraft changed title from Added flask support #70 to WIP: Added flask support #70 2022-10-19 21:08:13 +02:00
edraft added 1 commit 2022-10-19 22:42:57 +02:00
edraft added 1 commit 2022-10-20 09:47:29 +02:00
edraft added 1 commit 2022-10-20 12:57:05 +02:00
edraft added 1 commit 2022-10-20 12:57:39 +02:00
edraft added 8 commits 2022-10-20 20:49:41 +02:00
edraft changed title from WIP: Added flask support #70 to Added flask support #70 2022-10-20 20:49:53 +02:00
edraft added 1 commit 2022-10-20 21:19:31 +02:00
edraft added 1 commit 2022-10-21 12:45:49 +02:00
edraft added 2 commits 2022-10-21 13:50:16 +02:00
edraft added 1 commit 2022-10-21 19:25:21 +02:00
edraft added 1 commit 2022-10-21 23:01:13 +02:00
edraft changed title from Added flask support #70 to WIP: Added flask support #70 2022-10-21 23:01:24 +02:00
edraft reviewed 2022-10-21 23:01:58 +02:00
edraft left a comment
Author
Owner

Configs für api müssen noch angepasst/erstellt werden... :/

Configs für api müssen noch angepasst/erstellt werden... :/
edraft added 1 commit 2022-10-22 07:37:50 +02:00
edraft added 1 commit 2022-10-22 08:17:57 +02:00
edraft added 1 commit 2022-10-23 12:44:55 +02:00
edraft added 1 commit 2022-10-23 14:11:22 +02:00
edraft changed title from WIP: Added flask support #70 to Added flask support #70 2022-10-23 14:17:24 +02:00
edraft added 1 commit 2022-10-23 14:24:38 +02:00
edraft added 1 commit 2022-10-23 14:37:45 +02:00
edraft reviewed 2022-10-23 14:50:20 +02:00
edraft left a comment
Author
Owner

Client erkennt relativ schnell nen Disconnect auf kdb-test. Lokal kommt mir das nicht so vor.

Client erkennt relativ schnell nen Disconnect auf kdb-test. Lokal kommt mir das nicht so vor.
edraft added 1 commit 2022-10-23 15:05:04 +02:00
edraft added 1 commit 2022-10-23 15:11:08 +02:00
edraft added 1 commit 2022-10-23 15:32:07 +02:00
edraft added 1 commit 2022-10-23 15:37:43 +02:00
edraft added 1 commit 2022-10-23 16:35:27 +02:00
edraft added 1 commit 2022-10-23 16:38:51 +02:00
edraft added 1 commit 2022-10-23 17:49:35 +02:00
edraft added 2 commits 2022-10-23 18:24:35 +02:00
edraft added 1 commit 2022-10-23 22:41:27 +02:00
edraft changed title from Added flask support #70 to Added flask support #70 #75 2022-10-23 22:54:01 +02:00
edraft-dev approved these changes 2022-10-24 16:27:23 +02:00
edraft added 1 commit 2022-10-25 07:41:24 +02:00
edraft added 2 commits 2022-10-25 07:50:59 +02:00
edraft added 1 commit 2022-10-25 08:01:00 +02:00
edraft added 1 commit 2022-10-25 08:07:18 +02:00
edraft added 1 commit 2022-10-25 08:31:11 +02:00
edraft added 1 commit 2022-10-25 20:44:53 +02:00
edraft added 1 commit 2022-10-25 21:11:58 +02:00
edraft added 1 commit 2022-10-25 21:15:44 +02:00
edraft added 5 commits 2022-10-25 21:24:29 +02:00
edraft-dev approved these changes 2022-10-27 21:44:55 +02:00
Ebola-Chan requested changes 2022-11-03 22:10:38 +01:00
Ebola-Chan left a comment
Contributor

Wie schon mal erwähnt, Kommentare über die Versionsnummer können ignoriert werden.

Sonst bin ich der Meinungen dass bei allen SQL-Statements die ` durch ' ersetzt werden können.

Sonst mache ich mir noch Sorgen was ein eingeloggter User in der JSON alles mitgeben könnte. Diese Fälle konnte ich selber nicht prüfen, da ich keinen Erfolg mit Postman hatte.

In Postman habe ich die Requests bearbeitet und Variablen eingebaut. Du musst unter Enviroment einen eignen Enviroment anlegen und die Variablen "Host", "EMail" und "Password" anlegen.

Wie schon mal erwähnt, Kommentare über die Versionsnummer können ignoriert werden. Sonst bin ich der Meinungen dass bei allen SQL-Statements die ``` ` ``` durch ``` ' ``` ersetzt werden können. Sonst mache ich mir noch Sorgen was ein eingeloggter User in der JSON alles mitgeben könnte. Diese Fälle konnte ich selber nicht prüfen, da ich keinen Erfolg mit Postman hatte. In Postman habe ich die Requests bearbeitet und Variablen eingebaut. Du musst unter Enviroment einen eignen Enviroment anlegen und die Variablen "Host", "EMail" und "Password" anlegen.
@ -0,0 +70,4 @@
Console.write_line()
def is_restart(self):
return True if self._configuration.get_configuration('IS_RESTART') == 'true' else False #
Contributor

Reicht hier nicht ein

return self._configuration.get_configuration('IS_RESTART') == 'true'

oder ist das so ein Pythin ding dank der Typitisierung?

Reicht hier nicht ein ```python return self._configuration.get_configuration('IS_RESTART') == 'true' ``` oder ist das so ein Pythin ding dank der Typitisierung?
Author
Owner

Jain, gebe das als string rein, ist bisschen besser an der stelle, liegt an CPL

Jain, gebe das als string rein, ist bisschen besser an der stelle, liegt an CPL
edraft marked this conversation as resolved
@ -0,0 +8,4 @@
"LoggingSettings": {
"Path": "logs/",
"Filename": "bot.log",
"ConsoleLogLevel": "DEBUG",
Contributor

Log Level evtl wieder auch Trace setzen?

Log Level evtl wieder auch Trace setzen?
edraft marked this conversation as resolved
@ -0,0 +12,4 @@
self,
logger: ApiLogger,
api: Api,
feature_flags: FeatureFlagsSettings
Contributor

Muss hier dasd feature_flags nicht assigned werden?

Muss hier dasd feature_flags nicht assigned werden?
edraft marked this conversation as resolved
@ -0,0 +58,4 @@
@Route.get(f'{BasePath}/users/get/<email>')
@Route.authorize
async def get_user_from_email(self, email: str) -> Response:
Contributor

Was passiert, wenn eine andere E-Mail-Adresse angegeben wird?
Wenn eine positive Antwort gegeben wird: Ist es gewollt, dass ein User eine andere E-Mail-Adresse aufrufen kann?
Wenn ja: Soll ein User eine E-Mail-Adresse abfragen können, mit dem dieser nichts zu tun hat?

Was passiert, wenn eine andere E-Mail-Adresse angegeben wird? Wenn eine positive Antwort gegeben wird: Ist es gewollt, dass ein User eine andere E-Mail-Adresse aufrufen kann? Wenn ja: Soll ein User eine E-Mail-Adresse abfragen können, mit dem dieser nichts zu tun hat?
Author
Owner

was spricht dagegen?
So kannst du Profile von anderen usern sehen, kannst damit ja nichts ans pw kommen.

was spricht dagegen? So kannst du Profile von anderen usern sehen, kannst damit ja nichts ans pw kommen.
Contributor

Naja, so könnte sich jemand registrieren und dann somit eine Liste von gültigen E-Mails erstellen für Spammers.

Naja, so könnte sich jemand registrieren und dann somit eine Liste von gültigen E-Mails erstellen für Spammers.
Contributor

Wenn der User nur Profile von anderen Usern aufrufen kann, mit dem er zu tun hat, dann wäre es besser.

Wenn der User nur Profile von anderen Usern aufrufen kann, mit dem er zu tun hat, dann wäre es besser.
edraft marked this conversation as resolved
@ -0,0 +116,4 @@
@Route.post(f'{BasePath}/update-user')
@Route.authorize
async def update_user(self):
Contributor

Was passiert, wenn ein User eine JSON sendet mit der E-Mail-Adresse eines anderen Users?
Also versuchen die Einstellungen eines anderen Users überscheiben.

Was passiert, wenn ein User eine JSON sendet mit der E-Mail-Adresse eines anderen Users? Also versuchen die Einstellungen eines anderen Users überscheiben.
Author
Owner

Er muss das Passwort des Users kennen

Er muss das Passwort des Users kennen
edraft marked this conversation as resolved
@ -0,0 +46,4 @@
self._server_id = int(values['serverId'])
self._discord_id = int(values['discordId'])
self._name = values['name']
self._icon_url = int(values['iconURL'])
Contributor

Rückgabewert von

self._icon_url

wird in der Property als

Optional[str]

angegeben und hier mit

int(values['iconURL'])

zugewiesen.

Vielleicht klassischer Copy&Paste error?

Rückgabewert von ```python self._icon_url ``` wird in der Property als ```python Optional[str] ``` angegeben und hier mit ```python int(values['iconURL']) ``` zugewiesen. Vielleicht klassischer Copy&Paste error?
edraft marked this conversation as resolved
@ -0,0 +28,4 @@
@property
def str(self) -> str:
return f'{self._major}.{self._minor}.{self._micro}'
Contributor

Wenn ich mich recht entsinne, hattest du eine Mothode geschrieben, welche die Micro Nummer nicht mit ausgibt, wenn diese nicht gesetzt ist. Soll dies vielleicht auch hier eingebaut werden?

Wenn ich mich recht entsinne, hattest du eine Mothode geschrieben, welche die Micro Nummer nicht mit ausgibt, wenn diese nicht gesetzt ist. Soll dies vielleicht auch hier eingebaut werden?
Author
Owner

war in nem anderen kontext, könnte man hier auch machen, da sehe ich den Zweck aber nicht, da unser erster Release die 1.0.0 sein wird.

war in nem anderen kontext, könnte man hier auch machen, da sehe ich den Zweck aber nicht, da unser erster Release die 1.0.0 sein wird.
edraft marked this conversation as resolved
@ -0,0 +25,4 @@
cls._auth = auth
@classmethod
def authorize(cls, f: Callable = None, role: AuthRoleEnum = None):
Contributor

Vorschlag: Die HTTP-Response-Status-Codes in einem Enum packen, damit man schneller erkennen kann, um was die Response handelt.

Vorschlag: Die HTTP-Response-Status-Codes in einem Enum packen, damit man schneller erkennen kann, um was die Response handelt.
Author
Owner

Kann man später mal nachbessern aber die API ist jetzt noch übersichtlich genug

Kann man später mal nachbessern aber die API ist jetzt noch übersichtlich genug
edraft marked this conversation as resolved
@ -0,0 +15,4 @@
def upgrade(self):
self._logger.debug(__name__, 'Running upgrade')
self._cursor.execute(
Contributor

Was spricht dagegen einfache Anführungsstriche (') zu nutzen, statt `?

self._cursor.execute(
    str(f"""
    CREATE TABLE IF NOT EXISTS 'AuthUsers' (
        'Id' BIGINT NOT NULL AUTO_INCREMENT,
        'FirstName' VARCHAR(255),
        'LastName' VARCHAR(255),
        'EMail' VARCHAR(255),
        'Password' VARCHAR(255),
        'PasswordSalt' VARCHAR(255),
        'RefreshToken' VARCHAR(255),
        'ConfirmationId' VARCHAR(255) DEFAULT NULL,
        'ForgotPasswordId' VARCHAR(255) DEFAULT NULL,
        'OAuthId' VARCHAR(255) DEFAULT NULL,
        'RefreshTokenExpiryTime' DATETIME(6) NOT NULL,
        'AuthRole' INT NOT NULL DEFAULT 0,
        'CreatedAt' DATETIME(6) NOT NULL,
        'LastModifiedAt' DATETIME(6) NOT NULL,
        PRIMARY KEY('Id')
        );
        """)
Was spricht dagegen einfache Anführungsstriche (```'```) zu nutzen, statt ``` ` ```? ```python self._cursor.execute( str(f""" CREATE TABLE IF NOT EXISTS 'AuthUsers' ( 'Id' BIGINT NOT NULL AUTO_INCREMENT, 'FirstName' VARCHAR(255), 'LastName' VARCHAR(255), 'EMail' VARCHAR(255), 'Password' VARCHAR(255), 'PasswordSalt' VARCHAR(255), 'RefreshToken' VARCHAR(255), 'ConfirmationId' VARCHAR(255) DEFAULT NULL, 'ForgotPasswordId' VARCHAR(255) DEFAULT NULL, 'OAuthId' VARCHAR(255) DEFAULT NULL, 'RefreshTokenExpiryTime' DATETIME(6) NOT NULL, 'AuthRole' INT NOT NULL DEFAULT 0, 'CreatedAt' DATETIME(6) NOT NULL, 'LastModifiedAt' DATETIME(6) NOT NULL, PRIMARY KEY('Id') ); """) ```
Author
Owner

An sich nichts, arbeite da aber möglichst an dem was PhpMyAdmin mir als Code ausspucken würde. Da werden ` mehr als ' genutzt

An sich nichts, arbeite da aber möglichst an dem was PhpMyAdmin mir als Code ausspucken würde. Da werden ``` ` ``` mehr als ``` ' ``` genutzt
edraft marked this conversation as resolved
@ -0,0 +29,4 @@
`ForgotPasswordId` VARCHAR(255) DEFAULT NULL,
`OAuthId` VARCHAR(255) DEFAULT NULL,
`RefreshTokenExpiryTime` DATETIME(6) NOT NULL,
`AuthRole` INT NOT NULL DEFAULT '0',
Contributor

Warum ist der Defaultwert für ein Integer als ein String angegeben?

Warum ist der Defaultwert für ein Integer als ein String angegeben?
edraft marked this conversation as resolved
@ -0,0 +1,44 @@
{
"ProjectSettings": {
"Name": "admin",
"Version": {
Contributor

Muss version hier nicht 0.3.70dev sein?

Muss version hier nicht 0.3.70dev sein?
Ebola-Chan marked this conversation as resolved
@ -0,0 +1,44 @@
{
"ProjectSettings": {
"Name": "auto-role",
"Version": {
Contributor

Version 0.3.dev70?

Version 0.3.dev70?
Ebola-Chan marked this conversation as resolved
@ -0,0 +2,4 @@
"ProjectSettings": {
"Name": "base",
"Version": {
"Major": "1",
Contributor

Version 0.3.dev70?

Version 0.3.dev70?
Ebola-Chan marked this conversation as resolved
edraft added 1 commit 2022-11-05 11:52:03 +01:00
edraft added 1 commit 2022-11-05 11:52:34 +01:00
edraft added 1 commit 2022-11-05 12:03:25 +01:00
edraft added 1 commit 2022-11-05 12:39:52 +01:00
edraft added 1 commit 2022-11-05 12:40:24 +01:00
edraft added 1 commit 2022-11-05 12:42:25 +01:00
edraft added 1 commit 2022-11-05 13:49:05 +01:00
Ebola-Chan approved these changes 2022-11-05 13:53:11 +01:00
edraft-dev approved these changes 2022-11-05 13:55:06 +01:00
edraft merged commit 3516a164da into 0.3 2022-11-05 13:55:42 +01:00
edraft deleted branch #70 2022-11-05 13:55:43 +01:00
edraft removed this from the Sprint 0.3 milestone 2023-02-13 10:27:37 +01:00
Sign in to join this conversation.
No description provided.