summaryrefslogtreecommitdiffstats
path: root/store
diff options
context:
space:
mode:
authorJoram Wilander <jwawilander@gmail.com>2016-06-04 10:52:25 -0400
committerJoram Wilander <jwawilander@gmail.com>2016-06-04 10:52:25 -0400
commit971149d2b298b3408af7218c868ed0b3edd83c2e (patch)
tree86e082c9341ccd98bc6bf5894ce3ee7d14359550 /store
parent8d09c58b4dbfd9f9ad651e03263dc109d7680a47 (diff)
downloadchat-971149d2b298b3408af7218c868ed0b3edd83c2e.tar.gz
chat-971149d2b298b3408af7218c868ed0b3edd83c2e.tar.bz2
chat-971149d2b298b3408af7218c868ed0b3edd83c2e.zip
Don't allow users to be added to a channel they are not in the team of (#3246)
Diffstat (limited to 'store')
-rw-r--r--store/sql_channel_store.go8
-rw-r--r--store/sql_team_store.go8
2 files changed, 11 insertions, 5 deletions
diff --git a/store/sql_channel_store.go b/store/sql_channel_store.go
index 0a8873b57..3636fb62f 100644
--- a/store/sql_channel_store.go
+++ b/store/sql_channel_store.go
@@ -11,9 +11,9 @@ import (
)
const (
- MISSING_CHANNEL_ERROR = "store.sql_channel.get_by_name.missing.app_error"
- MISSING_MEMBER_ERROR = "store.sql_channel.get_member.missing.app_error"
- CHANNEL_EXISTS_ERROR = "store.sql_channel.save_channel.exists.app_error"
+ MISSING_CHANNEL_ERROR = "store.sql_channel.get_by_name.missing.app_error"
+ MISSING_CHANNEL_MEMBER_ERROR = "store.sql_channel.get_member.missing.app_error"
+ CHANNEL_EXISTS_ERROR = "store.sql_channel.save_channel.exists.app_error"
)
type SqlChannelStore struct {
@@ -579,7 +579,7 @@ func (s SqlChannelStore) GetMember(channelId string, userId string) StoreChannel
if err := s.GetReplica().SelectOne(&member, "SELECT * FROM ChannelMembers WHERE ChannelId = :ChannelId AND UserId = :UserId", map[string]interface{}{"ChannelId": channelId, "UserId": userId}); err != nil {
if err == sql.ErrNoRows {
- result.Err = model.NewLocAppError("SqlChannelStore.GetMember", MISSING_MEMBER_ERROR, nil, "channel_id="+channelId+"user_id="+userId+","+err.Error())
+ result.Err = model.NewLocAppError("SqlChannelStore.GetMember", MISSING_CHANNEL_MEMBER_ERROR, nil, "channel_id="+channelId+"user_id="+userId+","+err.Error())
} else {
result.Err = model.NewLocAppError("SqlChannelStore.GetMember", "store.sql_channel.get_member.app_error", nil, "channel_id="+channelId+"user_id="+userId+","+err.Error())
}
diff --git a/store/sql_team_store.go b/store/sql_team_store.go
index a9f265273..6e1deeb20 100644
--- a/store/sql_team_store.go
+++ b/store/sql_team_store.go
@@ -4,6 +4,8 @@
package store
import (
+ "database/sql"
+
"github.com/mattermost/platform/model"
"github.com/mattermost/platform/utils"
)
@@ -420,7 +422,11 @@ func (s SqlTeamStore) GetMember(teamId string, userId string) StoreChannel {
var member model.TeamMember
err := s.GetReplica().SelectOne(&member, "SELECT * FROM TeamMembers WHERE TeamId = :TeamId AND UserId = :UserId", map[string]interface{}{"TeamId": teamId, "UserId": userId})
if err != nil {
- result.Err = model.NewLocAppError("SqlTeamStore.GetMember", "store.sql_team.get_member.app_error", nil, "teamId="+teamId+" userId="+userId+" "+err.Error())
+ if err == sql.ErrNoRows {
+ result.Err = model.NewLocAppError("SqlTeamStore.GetMember", "store.sql_team.get_member.missing.app_error", nil, "teamId="+teamId+" userId="+userId+" "+err.Error())
+ } else {
+ result.Err = model.NewLocAppError("SqlTeamStore.GetMember", "store.sql_team.get_member.app_error", nil, "teamId="+teamId+" userId="+userId+" "+err.Error())
+ }
} else {
result.Data = member
}