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

removed "Secret" setting prevents plugin boot #1

Open
revolunet opened this issue Jun 4, 2019 · 5 comments · May be fixed by #2
Open

removed "Secret" setting prevents plugin boot #1

revolunet opened this issue Jun 4, 2019 · 5 comments · May be fixed by #2

Comments

@revolunet
Copy link

Looks like this setting has been removed by plugin.json here

But, on master branch, there is still a check on plugin startup on this param; this change prevents the plugin from installing and lead to :


  Must have secret key

  unable to start plugin: mattermost-google-calendar

  github.com/mattermost/mattermost-server/plugin.(*Environment).Activate

    /go/src/github.com/mattermost/mattermost-server/plugin/environment.go:216

  github.com/mattermost/mattermost-server/app.(*App).SyncPluginsActiveState

    /go/src/github.com/mattermost/mattermost-server/app/plugin.go:95

  github.com/mattermost/mattermost-server/app.(*App).InitPlugins.func2

    /go/src/github.com/mattermost/mattermost-server/app/plugin.go:172

  github.com/mattermost/mattermost-server/config.(*emitter).invokeConfigListeners.func1

    /go/src/github.com/mattermost/mattermost-server/config/emitter.go:35

  sync.(*Map).Range

    /usr/local/go/src/sync/map.go:333

  github.com/mattermost/mattermost-server/config.(*emitter).invokeConfigListeners

    /go/src/github.com/mattermost/mattermost-server/config/emitter.go:33

  github.com/mattermost/mattermost-server/config.(*commonStore).set

    /go/src/github.com/mattermost/mattermost-server/config/common.go:80

  github.com/mattermost/mattermost-server/config.(*FileStore).Set

    /go/src/github.com/mattermost/mattermost-server/config/file.go:97

  github.com/mattermost/mattermost-server/app.(*Server).UpdateConfig

    /go/src/github.com/mattermost/mattermost-server/app/config.go:53

  github.com/mattermost/mattermost-server/app.(*App).UpdateConfig

    /go/src/github.com/mattermost/mattermost-server/app/config.go:59

  github.com/mattermost/mattermost-server/app.(*App).EnablePlugin

    /go/src/github.com/mattermost/mattermost-server/app/plugin.go:245

  github.com/mattermost/mattermost-server/api4.enablePlugin

    /go/src/github.com/mattermost/mattermost-server/api4/plugin.go:187

  github.com/mattermost/mattermost-server/web.Handler.ServeHTTP

    /go/src/github.com/mattermost/mattermost-server/web/handlers.go:167

  github.com/mattermost/mattermost-server/vendor/github.com/gorilla/mux.(*Router).ServeHTTP

    /go/src/github.com/mattermost/mattermost-server/vendor/github.com/gorilla/mux/mux.go:162

  net/http.serverHandler.ServeHTTP

    /usr/local/go/src/net/http/server.go:2774

  net/http.(*conn).serve

    /usr/local/go/src/net/http/server.go:1878

  runtime.goexit

    /usr/local/go/src/runtime/asm_amd64.s:1337"
}
@unwmun-smobile
Copy link

I have the same problem.

@cinco
Copy link

cinco commented Nov 29, 2019

Looks like this setting has been removed by plugin.json here

But, on master branch, there is still a check on plugin startup on this param; this change prevents the plugin from installing and lead to :


  Must have secret key

  unable to start plugin: mattermost-google-calendar

  github.com/mattermost/mattermost-server/plugin.(*Environment).Activate

    /go/src/github.com/mattermost/mattermost-server/plugin/environment.go:216

  github.com/mattermost/mattermost-server/app.(*App).SyncPluginsActiveState

    /go/src/github.com/mattermost/mattermost-server/app/plugin.go:95

  github.com/mattermost/mattermost-server/app.(*App).InitPlugins.func2

    /go/src/github.com/mattermost/mattermost-server/app/plugin.go:172

  github.com/mattermost/mattermost-server/config.(*emitter).invokeConfigListeners.func1

    /go/src/github.com/mattermost/mattermost-server/config/emitter.go:35

  sync.(*Map).Range

    /usr/local/go/src/sync/map.go:333

  github.com/mattermost/mattermost-server/config.(*emitter).invokeConfigListeners

    /go/src/github.com/mattermost/mattermost-server/config/emitter.go:33

  github.com/mattermost/mattermost-server/config.(*commonStore).set

    /go/src/github.com/mattermost/mattermost-server/config/common.go:80

  github.com/mattermost/mattermost-server/config.(*FileStore).Set

    /go/src/github.com/mattermost/mattermost-server/config/file.go:97

  github.com/mattermost/mattermost-server/app.(*Server).UpdateConfig

    /go/src/github.com/mattermost/mattermost-server/app/config.go:53

  github.com/mattermost/mattermost-server/app.(*App).UpdateConfig

    /go/src/github.com/mattermost/mattermost-server/app/config.go:59

  github.com/mattermost/mattermost-server/app.(*App).EnablePlugin

    /go/src/github.com/mattermost/mattermost-server/app/plugin.go:245

  github.com/mattermost/mattermost-server/api4.enablePlugin

    /go/src/github.com/mattermost/mattermost-server/api4/plugin.go:187

  github.com/mattermost/mattermost-server/web.Handler.ServeHTTP

    /go/src/github.com/mattermost/mattermost-server/web/handlers.go:167

  github.com/mattermost/mattermost-server/vendor/github.com/gorilla/mux.(*Router).ServeHTTP

    /go/src/github.com/mattermost/mattermost-server/vendor/github.com/gorilla/mux/mux.go:162

  net/http.serverHandler.ServeHTTP

    /usr/local/go/src/net/http/server.go:2774

  net/http.(*conn).serve

    /usr/local/go/src/net/http/server.go:1878

  runtime.goexit

    /usr/local/go/src/runtime/asm_amd64.s:1337"
}

Did you ever compile and test?

@ybudimirov
Copy link

@revolunet
Were you able to finish setup of this plugin?

@revolunet
Copy link
Author

yes, setup finished ok but this plugin had other issues (cant remember) so we dont use it :/

@ybudimirov
Copy link

thank you for reply!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants