* don't report an error on plugin activation if already active * improved plugin logging events Log an error when a plugin's ServeHTTP fails, or when it unexpectedly terminates. Restart a plugin at most three times, allowing its failure to later bubble up under the "failed to stay running" status. * clarified plugin activation/deactivation Avoid repeatedly activating when any configuration bit changes. Improved logging. * constrain plugin ids to ^[a-zA-Z0-9-_\.]+$ and enforce minimum length Previously, the plugin id was used unsanitized to relocate the plugin bundle, which allowed writing outside the `plugins/` directory by using an `id` containing `../`. Similarly, an empty string was accepted as an id and led to unexpected error messages. * remove plugins by manifest path, not id If the id within the manifest ever diverges from the actual plugin location, it becomes impossible to remove via the API. Instead, if the plugin is found by id, remove the path containing the manifest. * ignore plugins with nil manifests If a plugin was detected, but had a manifest that couldn't be parsed, it will be left nil but still be listed among the packages. Skip over these in most cases to avoid segfaults. * leverage mlog more effectively for plugins * build issues
274 строки
7.2 KiB
Go
274 строки
7.2 KiB
Go
// Copyright (c) 2017-present Mattermost, Inc. All Rights Reserved.
|
|
// See License.txt for license information.
|
|
|
|
package rpcplugintest
|
|
|
|
import (
|
|
"encoding/json"
|
|
"fmt"
|
|
"io/ioutil"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"os"
|
|
"path/filepath"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/mock"
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"github.com/mattermost/mattermost-server/model"
|
|
"github.com/mattermost/mattermost-server/plugin"
|
|
"github.com/mattermost/mattermost-server/plugin/plugintest"
|
|
)
|
|
|
|
type SupervisorProviderFunc = func(*model.BundleInfo) (plugin.Supervisor, error)
|
|
|
|
func TestSupervisorProvider(t *testing.T, sp SupervisorProviderFunc) {
|
|
for name, f := range map[string]func(*testing.T, SupervisorProviderFunc){
|
|
"Supervisor": testSupervisor,
|
|
"Supervisor_InvalidExecutablePath": testSupervisor_InvalidExecutablePath,
|
|
"Supervisor_NonExistentExecutablePath": testSupervisor_NonExistentExecutablePath,
|
|
"Supervisor_StartTimeout": testSupervisor_StartTimeout,
|
|
"Supervisor_PluginCrash": testSupervisor_PluginCrash,
|
|
"Supervisor_PluginRepeatedlyCrash": testSupervisor_PluginRepeatedlyCrash,
|
|
} {
|
|
t.Run(name, func(t *testing.T) { f(t, sp) })
|
|
}
|
|
}
|
|
|
|
func testSupervisor(t *testing.T, sp SupervisorProviderFunc) {
|
|
dir, err := ioutil.TempDir("", "")
|
|
require.NoError(t, err)
|
|
defer os.RemoveAll(dir)
|
|
|
|
backend := filepath.Join(dir, "backend.exe")
|
|
CompileGo(t, `
|
|
package main
|
|
|
|
import (
|
|
"github.com/mattermost/mattermost-server/plugin/rpcplugin"
|
|
)
|
|
|
|
type MyPlugin struct {}
|
|
|
|
func main() {
|
|
rpcplugin.Main(&MyPlugin{})
|
|
}
|
|
`, backend)
|
|
|
|
ioutil.WriteFile(filepath.Join(dir, "plugin.json"), []byte(`{"id": "foo", "backend": {"executable": "backend.exe"}}`), 0600)
|
|
|
|
bundle := model.BundleInfoForPath(dir)
|
|
supervisor, err := sp(bundle)
|
|
require.NoError(t, err)
|
|
require.NoError(t, supervisor.Start(nil))
|
|
require.NoError(t, supervisor.Stop())
|
|
}
|
|
|
|
func testSupervisor_InvalidExecutablePath(t *testing.T, sp SupervisorProviderFunc) {
|
|
dir, err := ioutil.TempDir("", "")
|
|
require.NoError(t, err)
|
|
defer os.RemoveAll(dir)
|
|
|
|
ioutil.WriteFile(filepath.Join(dir, "plugin.json"), []byte(`{"id": "foo", "backend": {"executable": "/foo/../../backend.exe"}}`), 0600)
|
|
|
|
bundle := model.BundleInfoForPath(dir)
|
|
supervisor, err := sp(bundle)
|
|
assert.Nil(t, supervisor)
|
|
assert.Error(t, err)
|
|
}
|
|
|
|
func testSupervisor_NonExistentExecutablePath(t *testing.T, sp SupervisorProviderFunc) {
|
|
dir, err := ioutil.TempDir("", "")
|
|
require.NoError(t, err)
|
|
defer os.RemoveAll(dir)
|
|
|
|
ioutil.WriteFile(filepath.Join(dir, "plugin.json"), []byte(`{"id": "foo", "backend": {"executable": "thisfileshouldnotexist"}}`), 0600)
|
|
|
|
bundle := model.BundleInfoForPath(dir)
|
|
supervisor, err := sp(bundle)
|
|
require.NotNil(t, supervisor)
|
|
require.NoError(t, err)
|
|
|
|
require.Error(t, supervisor.Start(nil))
|
|
}
|
|
|
|
// If plugin development goes really wrong, let's make sure plugin activation won't block forever.
|
|
func testSupervisor_StartTimeout(t *testing.T, sp SupervisorProviderFunc) {
|
|
dir, err := ioutil.TempDir("", "")
|
|
require.NoError(t, err)
|
|
defer os.RemoveAll(dir)
|
|
|
|
backend := filepath.Join(dir, "backend.exe")
|
|
CompileGo(t, `
|
|
package main
|
|
|
|
func main() {
|
|
for {
|
|
}
|
|
}
|
|
`, backend)
|
|
|
|
ioutil.WriteFile(filepath.Join(dir, "plugin.json"), []byte(`{"id": "foo", "backend": {"executable": "backend.exe"}}`), 0600)
|
|
|
|
bundle := model.BundleInfoForPath(dir)
|
|
supervisor, err := sp(bundle)
|
|
require.NoError(t, err)
|
|
require.Error(t, supervisor.Start(nil))
|
|
}
|
|
|
|
// Crashed plugins should be relaunched.
|
|
func testSupervisor_PluginCrash(t *testing.T, sp SupervisorProviderFunc) {
|
|
dir, err := ioutil.TempDir("", "")
|
|
require.NoError(t, err)
|
|
defer os.RemoveAll(dir)
|
|
|
|
backend := filepath.Join(dir, "backend.exe")
|
|
CompileGo(t, `
|
|
package main
|
|
|
|
import (
|
|
"os"
|
|
|
|
"github.com/mattermost/mattermost-server/plugin"
|
|
"github.com/mattermost/mattermost-server/plugin/rpcplugin"
|
|
)
|
|
|
|
type Configuration struct {
|
|
ShouldExit bool
|
|
}
|
|
|
|
type MyPlugin struct {
|
|
config Configuration
|
|
}
|
|
|
|
func (p *MyPlugin) OnActivate(api plugin.API) error {
|
|
api.LoadPluginConfiguration(&p.config)
|
|
return nil
|
|
}
|
|
|
|
func (p *MyPlugin) OnDeactivate() error {
|
|
if p.config.ShouldExit {
|
|
os.Exit(1)
|
|
}
|
|
return nil
|
|
}
|
|
|
|
func main() {
|
|
rpcplugin.Main(&MyPlugin{})
|
|
}
|
|
`, backend)
|
|
|
|
ioutil.WriteFile(filepath.Join(dir, "plugin.json"), []byte(`{"id": "foo", "backend": {"executable": "backend.exe"}}`), 0600)
|
|
|
|
var api plugintest.API
|
|
shouldExit := true
|
|
api.On("LoadPluginConfiguration", mock.MatchedBy(func(x interface{}) bool { return true })).Return(func(dest interface{}) error {
|
|
err := json.Unmarshal([]byte(fmt.Sprintf(`{"ShouldExit": %v}`, shouldExit)), dest)
|
|
shouldExit = false
|
|
return err
|
|
})
|
|
|
|
bundle := model.BundleInfoForPath(dir)
|
|
supervisor, err := sp(bundle)
|
|
require.NoError(t, err)
|
|
require.NoError(t, supervisor.Start(&api))
|
|
|
|
failed := false
|
|
recovered := false
|
|
for i := 0; i < 30; i++ {
|
|
if supervisor.Hooks().OnDeactivate() == nil {
|
|
require.True(t, failed)
|
|
recovered = true
|
|
break
|
|
} else {
|
|
failed = true
|
|
}
|
|
time.Sleep(time.Millisecond * 100)
|
|
}
|
|
assert.True(t, recovered)
|
|
require.NoError(t, supervisor.Stop())
|
|
}
|
|
|
|
// Crashed plugins should be relaunched at most three times.
|
|
func testSupervisor_PluginRepeatedlyCrash(t *testing.T, sp SupervisorProviderFunc) {
|
|
dir, err := ioutil.TempDir("", "")
|
|
require.NoError(t, err)
|
|
defer os.RemoveAll(dir)
|
|
|
|
backend := filepath.Join(dir, "backend.exe")
|
|
CompileGo(t, `
|
|
package main
|
|
|
|
import (
|
|
"net/http"
|
|
"os"
|
|
|
|
"github.com/mattermost/mattermost-server/plugin/rpcplugin"
|
|
)
|
|
|
|
type MyPlugin struct {
|
|
crashing bool
|
|
}
|
|
|
|
func (p *MyPlugin) ServeHTTP(w http.ResponseWriter, r *http.Request) {
|
|
if r.Method == http.MethodPost {
|
|
p.crashing = true
|
|
go func() {
|
|
os.Exit(1)
|
|
}()
|
|
}
|
|
|
|
if p.crashing {
|
|
w.WriteHeader(http.StatusInternalServerError)
|
|
} else {
|
|
w.WriteHeader(http.StatusOK)
|
|
}
|
|
}
|
|
|
|
func main() {
|
|
rpcplugin.Main(&MyPlugin{})
|
|
}
|
|
`, backend)
|
|
|
|
ioutil.WriteFile(filepath.Join(dir, "plugin.json"), []byte(`{"id": "foo", "backend": {"executable": "backend.exe"}}`), 0600)
|
|
|
|
var api plugintest.API
|
|
bundle := model.BundleInfoForPath(dir)
|
|
supervisor, err := sp(bundle)
|
|
require.NoError(t, err)
|
|
require.NoError(t, supervisor.Start(&api))
|
|
|
|
for attempt := 1; attempt <= 4; attempt++ {
|
|
// Verify that the plugin is operational
|
|
response := httptest.NewRecorder()
|
|
supervisor.Hooks().ServeHTTP(response, httptest.NewRequest(http.MethodGet, "/plugins/id", nil))
|
|
require.Equal(t, http.StatusOK, response.Result().StatusCode)
|
|
|
|
// Crash the plugin
|
|
supervisor.Hooks().ServeHTTP(httptest.NewRecorder(), httptest.NewRequest(http.MethodPost, "/plugins/id", nil))
|
|
|
|
// Wait for it to potentially recover
|
|
recovered := false
|
|
for i := 0; i < 125; i++ {
|
|
response := httptest.NewRecorder()
|
|
supervisor.Hooks().ServeHTTP(response, httptest.NewRequest(http.MethodGet, "/plugins/id", nil))
|
|
if response.Result().StatusCode == http.StatusOK {
|
|
recovered = true
|
|
break
|
|
}
|
|
|
|
time.Sleep(time.Millisecond * 100)
|
|
}
|
|
|
|
if attempt < 4 {
|
|
require.True(t, recovered, "failed to recover after attempt %d", attempt)
|
|
} else {
|
|
require.False(t, recovered, "unexpectedly recovered after attempt %d", attempt)
|
|
}
|
|
}
|
|
require.NoError(t, supervisor.Stop())
|
|
}
|