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.

Sign up for GitHub

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

Jump to bottom

Do not limit messages when using maildir storage driver #178

Open
sabenok opened this issue Nov 3, 2017 · 0 comments
Open

Do not limit messages when using maildir storage driver #178

sabenok opened this issue Nov 3, 2017 · 0 comments
Labels
1.1 bug
Milestone
v1.1

Comments

@sabenok
Copy link

sabenok commented Nov 3, 2017

When using maildir storage and executing the request api/v2/messages?limit=50 all messages are returned, not specified in the limit.
The problem can be seen in the maildir storage code https://github.com/mailhog/storage/blob/master/maildir.go#L127
dsalahutdinov added a commit to dsalahutdinov/storage that referenced this issue Nov 3, 2017
@dsalahutdinov
fix: paginate messages for maildir storage
0ddbe7e 
Fixes mailhog/MailHog#178
@dsalahutdinov dsalahutdinov mentioned this issue Nov 3, 2017
Open
dsalahutdinov added a commit to dsalahutdinov/storage that referenced this issue Nov 3, 2017
@dsalahutdinov
fix: paginate messages for maildir storage
bdeece4 
Fixes mailhog/MailHog#178
dsalahutdinov added a commit to dsalahutdinov/storage that referenced this issue Nov 3, 2017
@dsalahutdinov
fix: paginate messages for maildir storage
44935ad 
Fixes mailhog/MailHog#178
Open
@tyndyll tyndyll added this to the v1.1 milestone Aug 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
1.1 bug
Projects
None yet
Milestone
v1.1
Development

No branches or pull requests
2 participants
@tyndyll @sabenok