diff options
| author | Joram Wilander <jwawilander@gmail.com> | 2016-06-04 10:52:25 -0400 |
|---|---|---|
| committer | Joram Wilander <jwawilander@gmail.com> | 2016-06-04 10:52:25 -0400 |
| commit | 971149d2b298b3408af7218c868ed0b3edd83c2e (patch) | |
| tree | 86e082c9341ccd98bc6bf5894ce3ee7d14359550 /store/sql_team_store.go | |
| parent | 8d09c58b4dbfd9f9ad651e03263dc109d7680a47 (diff) | |
| download | chat-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/sql_team_store.go')
| -rw-r--r-- | store/sql_team_store.go | 8 |
1 files changed, 7 insertions, 1 deletions
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 } |