No reviewers
Labels
No Label
MVP
web-interface
bug
bugfix
cli
discussion
documentation
duplicate
expense
high
expense
low
expense
medium
expense
very-high
help wanted
improvement
invalid
priority
blocker
priority
high
priority
low
priority
medium
question
refactoring
release
story
wontfix
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: sh-edraft.de/sh_discord_bot#71
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "#70"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
#70
#72
#75
#83
#85
#86
Added flask support #70to WIP: Added flask support #7098c1f40024
to029b46d7de
e564956813
to651482a1b9
WIP: Added flask support #70to Added flask support #70Added flask support #70to WIP: Added flask support #70WIP: Added flask support #70to Added flask support #70Added flask support #70to WIP: Added flask support #70WIP: Added flask support #70to Added flask support #70Added flask support #70to WIP: Added flask support #70Configs für api müssen noch angepasst/erstellt werden... :/
WIP: Added flask support #70to Added flask support #70Client erkennt relativ schnell nen Disconnect auf kdb-test. Lokal kommt mir das nicht so vor.
Added flask support #70to Added flask support #70 #75Wie 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 #
Reicht hier nicht ein
oder ist das so ein Pythin ding dank der Typitisierung?
Jain, gebe das als string rein, ist bisschen besser an der stelle, liegt an CPL
@ -0,0 +8,4 @@
"LoggingSettings": {
"Path": "logs/",
"Filename": "bot.log",
"ConsoleLogLevel": "DEBUG",
Log Level evtl wieder auch Trace setzen?
@ -0,0 +12,4 @@
self,
logger: ApiLogger,
api: Api,
feature_flags: FeatureFlagsSettings
Muss hier dasd feature_flags nicht assigned werden?
@ -0,0 +58,4 @@
@Route.get(f'{BasePath}/users/get/<email>')
@Route.authorize
async def get_user_from_email(self, email: str) -> Response:
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 spricht dagegen?
So kannst du Profile von anderen usern sehen, kannst damit ja nichts ans pw kommen.
Naja, so könnte sich jemand registrieren und dann somit eine Liste von gültigen E-Mails erstellen für Spammers.
Wenn der User nur Profile von anderen Usern aufrufen kann, mit dem er zu tun hat, dann wäre es besser.
@ -0,0 +116,4 @@
@Route.post(f'{BasePath}/update-user')
@Route.authorize
async def update_user(self):
Was passiert, wenn ein User eine JSON sendet mit der E-Mail-Adresse eines anderen Users?
Also versuchen die Einstellungen eines anderen Users überscheiben.
Er muss das Passwort des Users kennen
@ -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'])
Rückgabewert von
wird in der Property als
angegeben und hier mit
zugewiesen.
Vielleicht klassischer Copy&Paste error?
@ -0,0 +28,4 @@
@property
def str(self) -> str:
return f'{self._major}.{self._minor}.{self._micro}'
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?
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.
@ -0,0 +25,4 @@
cls._auth = auth
@classmethod
def authorize(cls, f: Callable = None, role: AuthRoleEnum = None):
Vorschlag: Die HTTP-Response-Status-Codes in einem Enum packen, damit man schneller erkennen kann, um was die Response handelt.
Kann man später mal nachbessern aber die API ist jetzt noch übersichtlich genug
@ -0,0 +15,4 @@
def upgrade(self):
self._logger.debug(__name__, 'Running upgrade')
self._cursor.execute(
Was spricht dagegen einfache Anführungsstriche (
'
) zu nutzen, statt`
?An sich nichts, arbeite da aber möglichst an dem was PhpMyAdmin mir als Code ausspucken würde. Da werden
`
mehr als'
genutzt@ -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',
Warum ist der Defaultwert für ein Integer als ein String angegeben?
@ -0,0 +1,44 @@
{
"ProjectSettings": {
"Name": "admin",
"Version": {
Muss version hier nicht 0.3.70dev sein?
@ -0,0 +1,44 @@
{
"ProjectSettings": {
"Name": "auto-role",
"Version": {
Version 0.3.dev70?
@ -0,0 +2,4 @@
"ProjectSettings": {
"Name": "base",
"Version": {
"Major": "1",
Version 0.3.dev70?