Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The limit set for a filter in a room's StateFilter is ignored #7306

Open
matrixbot opened this issue Dec 17, 2023 · 0 comments
Open

The limit set for a filter in a room's StateFilter is ignored #7306

matrixbot opened this issue Dec 17, 2023 · 0 comments

Comments

@matrixbot
Copy link
Collaborator

matrixbot commented Dec 17, 2023

This issue has been migrated from #7306.


Description

I'm using the below filter in a sync:

{
  "room": {
    "state": {
      "types": [
        "m.room.member",
        "m.room.name"
      ],
      "limit": 10
    },
    "timeline": {
      "limit": 10
    },
    "ephemeral": {
      "not_types": [ "*" ]
    }
  }
}

Please note that both room state and timeline limits are set to 10. In the resulting JSON I see my limit applied in timeline events (also limited: true), but not in room state events (one room in join has 4000+ events listed). According to the client-server API docs a limit clause should apply to StateFilter

Steps to reproduce

  • perform a sync without since
  • include a filter in the query that requests room.state.limit = 10

The limit should apply in e.g. rooms.join.<roomId>.state.events, but it doesn't seem to do so.

  • Homeserver: mozilla.modular.im

If not matrix.org:

  • Version: {"server_version": "1.12.3", "python_version": "3.7.7"}

  • Install method:

  • Platform:
@matrixbot matrixbot changed the title Dummy issue The limit set for a filter in a room's StateFilter is ignored Dec 21, 2023
@matrixbot matrixbot reopened this Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant