[GH-618]: Added slack attachment for webhook posts, in order to comment, edit, and close issues.

server/plugin/api_test.go

@@ -127,7 +127,7 @@ func TestPlugin_ServeHTTP(t *testing.T) {
 			p.SetAPI(&plugintest.API{})
 
 			req := test.httpTest.CreateHTTPRequest(test.request)
-			req.Header.Add("Mattermost-User-ID", test.userID)
+			req.Header.Add(headerMattermostUserID, test.userID)
 			rr := httptest.NewRecorder()
 			p.ServeHTTP(&plugin.Context{}, rr, req)
 			test.httpTest.CompareHTTPResponse(rr, test.expectedResponse)
@@ -356,7 +356,7 @@ func TestGetGitHubUser(t *testing.T) {
 
 				var response APIErrorResponse
 				_ = json.NewDecoder(rec.Body).Decode(&response)
-				assert.Contains(t, response.Message, "Please provide a JSON object.")
+				assert.Contains(t, response.Message, "Please provide a valid JSON object.")
 			},
 		},
 		{

server/plugin/mm_34646_token_refresh.go

@@ -115,7 +115,7 @@ func (p *Plugin) forceResetUserTokenMM34646(ctx context.Context, config *Configu
 	info.MM34646ResetTokenDone = true
 	err = p.storeGitHubUserInfo(info)
 	if err != nil {
-		return "", errors.Wrap(err, "failed to store updated GitHubUserInfo")
+		return "", errors.Wrap(err, "failed to store updated serializer.GitHubUserInfo")
 	}
 	p.client.Log.Debug("Updated user access token for MM-34646", "user_id", info.UserID)
 

server/plugin/oauth.go

@@ -5,13 +5,58 @@ package plugin
 
 import (
 	"sync"
+
+	"golang.org/x/oauth2"
 )
 
+type GitHubUserRequest struct {
+	UserID string `json:"user_id"`
+}
+
+type GitHubUserResponse struct {
+	Username string `json:"username"`
+}
+
+type ConnectedResponse struct {
+	Connected           bool                   `json:"connected"`
+	GitHubUsername      string                 `json:"github_username"`
+	GitHubClientID      string                 `json:"github_client_id"`
+	EnterpriseBaseURL   string                 `json:"enterprise_base_url,omitempty"`
+	Organizations       []string               `json:"organizations"`
+	UserSettings        *UserSettings          `json:"user_settings"`
+	ClientConfiguration map[string]interface{} `json:"configuration"`
+}
+
+type UserSettings struct {
+	SidebarButtons        string `json:"sidebar_buttons"`
+	DailyReminder         bool   `json:"daily_reminder"`
+	DailyReminderOnChange bool   `json:"daily_reminder_on_change"`
+	Notifications         bool   `json:"notifications"`
+}
+
+type GitHubUserInfo struct {
+	UserID              string
+	Token               *oauth2.Token
+	GitHubUsername      string
+	LastToDoPostAt      int64
+	Settings            *UserSettings
+	AllowedPrivateRepos bool
+
+	// MM34646ResetTokenDone is set for a user whose token has been reset for MM-34646.
+	MM34646ResetTokenDone bool
+}
+
 type OAuthCompleteEvent struct {
 	UserID string
 	Err    error
 }
 
+type OAuthState struct {
+	UserID         string `json:"user_id"`
+	Token          string `json:"token"`
+	PrivateAllowed bool   `json:"private_allowed"`
+}
+
 type OAuthBroker struct {
 	sendOAuthCompleteEvent func(event OAuthCompleteEvent)
 

server/plugin/plugin.go

@@ -13,6 +13,7 @@ import (
 	"regexp"
 	"strings"
 	"sync"
+	"time"
 
 	"github.com/google/go-github/v54/github"
 	"github.com/gorilla/mux"
@@ -30,10 +31,14 @@ import (
 )
 
 const (
-	githubTokenKey       = "_githubtoken"
-	githubOauthKey       = "githuboauthkey_"
-	githubUsernameKey    = "_githubusername"
-	githubPrivateRepoKey = "_githubprivate"
+	githubTokenKey                    = "_githubtoken"
+	githubOauthKey                    = "githuboauthkey_"
+	githubUsernameKey                 = "_githubusername"
+	githubPrivateRepoKey              = "_githubprivate"
+	githubObjectTypeIssue             = "issue"
+	githubObjectTypeIssueComment      = "issue_comment"
+	githubObjectTypePRReviewComment   = "pr_review_comment"
+	githubObjectTypeDiscussionComment = "discussion_comment"
 
 	mm34646MutexKey = "mm34646_token_reset_mutex"
 	mm34646DoneKey  = "mm34646_token_reset_done"
@@ -59,6 +64,39 @@ const (
 
 	chimeraGitHubAppIdentifier = "plugin-github"
 
+	apiErrorIDNotConnected = "not_connected"
+
+	// TokenTTL is the OAuth token expiry duration in seconds
+	tokenTTL = 600
+
+	requestTimeout         = 30 * time.Second
+	oauthCompleteTimeout   = 2 * time.Minute
+	headerMattermostUserID = "Mattermost-User-ID"
+	ownerQueryParam        = "owner"
+	repoQueryParam         = "repo"
+	numberQueryParam       = "number"
+	postIDQueryParam       = "postId"
+	channelIDParam         = "channelId"
+
+	issueStatus         = "status"
+	assigneesForProps   = "assignees"
+	labelsForProps      = "labels"
+	descriptionForProps = "description"
+	titleForProps       = "title"
+	attachmentsForProps = "attachments"
+	issueNumberForProps = "issue_number"
+	issueURLForProps    = "issue_url"
+	repoOwnerForProps   = "repo_owner"
+	repoNameForProps    = "repo_name"
+
+	statusClose  = "Close"
+	statusReopen = "Reopen"
+
+	issueCompleted  = "completed"
+	issueNotPlanned = "not_planned"
+	issueClose      = "closed"
+	issueOpen       = "open"
+
 	invalidTokenError = "401 Bad credentials" //#nosec G101 -- False positive
 )
 
@@ -604,25 +642,6 @@ func (p *Plugin) getOAuthConfigForChimeraApp(scopes []string) (*oauth2.Config, e
 	}, nil
 }
 
-type GitHubUserInfo struct {
-	UserID              string
-	Token               *oauth2.Token
-	GitHubUsername      string
-	LastToDoPostAt      int64
-	Settings            *UserSettings
-	AllowedPrivateRepos bool
-
-	// MM34646ResetTokenDone is set for a user whose token has been reset for MM-34646.
-	MM34646ResetTokenDone bool
-}
-
-type UserSettings struct {
-	SidebarButtons        string `json:"sidebar_buttons"`
-	DailyReminder         bool   `json:"daily_reminder"`
-	DailyReminderOnChange bool   `json:"daily_reminder_on_change"`
-	Notifications         bool   `json:"notifications"`
-}
-
 func (p *Plugin) storeGitHubUserInfo(info *GitHubUserInfo) error {
 	config := p.getConfiguration()
 
@@ -1123,7 +1142,7 @@ func (p *Plugin) sendRefreshEvent(userID string) {
 		return
 	}
 
-	contentMap, err := sidebarContent.toMap()
+	contentMap, err := sidebarContent.ToMap()
 	if err != nil {
 		p.client.Log.Warn("Failed to convert sidebar content to map", "error", err.Error())
 		return
@@ -1136,7 +1155,7 @@ func (p *Plugin) sendRefreshEvent(userID string) {
 	)
 }
 
-func (s *SidebarContent) toMap() (map[string]interface{}, error) {
+func (s *SidebarContent) ToMap() (map[string]interface{}, error) {
 	var m map[string]interface{}
 	bytes, err := json.Marshal(&s)
 	if err != nil {
@@ -1150,6 +1169,10 @@ func (s *SidebarContent) toMap() (map[string]interface{}, error) {
 	return m, nil
 }
 
+func (e *APIErrorResponse) Error() string {
+	return e.Message
+}
+
 // getUsername returns the GitHub username for a given Mattermost user,
 // if the user is connected to GitHub via this plugin.
 // Otherwise it return the Mattermost username. It will be escaped via backticks.
@@ -1171,6 +1194,10 @@ func (p *Plugin) getUsername(mmUserID string) (string, error) {
 	return "@" + info.GitHubUsername, nil
 }
 
+func (p *Plugin) GetPluginAPIPath() string {
+	return fmt.Sprintf("%s/plugins/%s/api/v1", *p.client.Configuration.GetConfig().ServiceSettings.SiteURL, Manifest.Id)
+}
+
 func (p *Plugin) useGitHubClient(info *GitHubUserInfo, toRun func(info *GitHubUserInfo, token *oauth2.Token) error) error {
 	err := toRun(info, info.Token)
 	if err != nil {

server/plugin/test_utils.go

@@ -25,10 +25,6 @@ const (
 	MockPostMessage = "mockPostMessage"
 )
 
-type GitHubUserResponse struct {
-	Username string `json:"username"`
-}
-
 func GetMockGHUserInfo(p *Plugin) (*GitHubUserInfo, error) {
 	encryptionKey := "dummyEncryptKey1"
 	p.setConfiguration(&Configuration{EncryptionKey: encryptionKey})