Skip to content

Conversation

@nickvergessen
Copy link
Member

It's a misbehaving bug in the frontend but we are yet unsure how it occures

๐Ÿ› ๏ธ API Checklist

๐Ÿ Checklist

  • โ›‘๏ธ Tests (unit and/or integration) are included or not possible
  • ๐Ÿ“˜ API documentation in docs/ has been updated or is not required
  • ๐Ÿ”– Capability is added or not needed

@nickvergessen nickvergessen added 3. to review bug feature: api ๐Ÿ› ๏ธ OCS API for conversations, chats and participants labels Apr 9, 2024
@nickvergessen nickvergessen self-assigned this Apr 9, 2024
@nickvergessen
Copy link
Member Author

/backport to stable29

It's a misbehaving bug in the frontend but we are yet unsure how it occurres

Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen nickvergessen force-pushed the bugfix/noid/dont-bruteforce-throttle-undefined branch from b235337 to 2dfddf9 Compare April 9, 2024 10:50
@DorraJaouad
Copy link
Contributor

we are yet unsure how it occures

switching between rooms too fast ?

@nickvergessen
Copy link
Member Author

Well the problem is it's not an API call, but actually requesting the frontend endpoint index.php/call/undefined

@Antreesy
Copy link
Contributor

Antreesy commented Apr 9, 2024

index.php/call/undefined

Could we track it somehow? Do we know the method (GET, POST, PUT, DELETE) where it happens?

@nickvergessen
Copy link
Member Author

I only see 3 technical options:

  1. Dashboard:
    callLink() {
    return (conversation) => {
    return generateUrl('call/' + conversation.token)
    }
    },
    but it was not accessed by the user today
  2. Signaling:
    window.location = generateUrl('call/' + token)
    but the user did not join a call today
  3. Talk Hash:

    spreed/src/App.vue

    Lines 412 to 416 in d89d6f2

    if (this.isNextcloudTalkHashDirty) {
    // Nextcloud Talk configuration changed, reload the page when changing configuration
    window.location = generateUrl('call/' + to.params.token)
    return
    }
    But the hash was not changed 8 times today. There was an update last night, so that could be the main root cause, but then it should be once (per tab?) and even then I would not know why to.params.token would be empty

@DorraJaouad
Copy link
Contributor

the frontend endpoint index.php/call/undefined

Maybe switching between rooms too fast leads to token === undefined . Because I see not found page rendered so token must be faulty.

@nickvergessen
Copy link
Member Author

With fast changing the only thing I can reproduce is ending up on "room not found" index.php/apps/spreed/not-found but never on call/undefined

@nickvergessen
Copy link
Member Author

Today's user confirmed they only worked in mostly the same room (also what I see in the logs), but toggled between shared attachments and the chat frequently to leave comments about it

@SystemKeeper
Copy link
Contributor

Does the call always result in a โ€žnot-foundโ€œ page in talk? Do we know it comes from talk itself or could it be a notification with a wrong action or something?

@nickvergessen
Copy link
Member Author

Does the call always result in a โ€žnot-foundโ€œ page in talk?

Well the conversation with the token undefined does not exist, so any follow up request should lead to that. But it's the frontpage, so it will just load the "home".

Do we know it comes from talk itself or could it be a notification with a wrong action or something?

Persons didn't report any clicking on notifications. They opened attachments from the chat and closed them again.
I can reproduce the access.log pattern with that description apart from the call/undefined

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug feature: api ๐Ÿ› ๏ธ OCS API for conversations, chats and participants

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants