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

Additions to the REST API #7793

Merged
merged 6 commits into from
Aug 22, 2017
Merged

Additions to the REST API #7793

merged 6 commits into from
Aug 22, 2017

Conversation

graywolf336
Copy link
Contributor

@rafaelks here are a few additions for the mobile apps. Once you find out more information regarding the oAuth items I will start working on getting them added.

Closes #7775 4f3c9d4
Closes #7765 fb04102
Closes #7764 fb04102
Closes #7763 ae87222 /api/v1/channels.messages?roomId=GENERAL&query={ "pinned": true }
Closes #7762 ae87222 /api/v1/channels.messages?roomId=GENERAL&query={ "starred._id": { "$in": ["${userid}"] } }
Closes #7761 1914d7a
Closes #7760 25792e6
Closes #7770 ae3f821
Closes #7241 ae3f821
Closes #7262 ae87222

In addition to the items above, direct message endpoints no longer require a user to know the room's id. Instead you can now pass the query parameter username to the endpoints and it will assume you mean a direct message conversation between the authenticated user and the username provided.

…oints. Rework the direct messages endpoints to support username instead of room id
1. process_incoming_request can now return falsey (null, undefined, 0, false, etc) and nothing will be sent to the channel
2. integrations can post messages to channels without the user being in them (direct messages, etc)
3. users can't brute force check if a room exists by successfully sending a message there
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-7793 August 18, 2017 08:43 Inactive
@rafaelks
Copy link
Contributor

Looking good @graywolf336! Thank you! 👍 👏

@engelgabriel engelgabriel added this to the 0.59.0 milestone Aug 18, 2017
@rodrigok rodrigok merged commit 2b79fa1 into develop Aug 22, 2017
@rodrigok rodrigok deleted the rest-api-for-native-apps branch August 22, 2017 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment