From 580b546862860ca389305d0d4614471095ec67fe Mon Sep 17 00:00:00 2001 From: Jesse Hallam Date: Wed, 3 Oct 2018 13:13:19 -0400 Subject: MM-12193: remove auto unmarshalling (#9519) * MM-12193: remove auto configuration unmarshalling Since plugin hook events are called concurrently, there's no way for the plugin framework to coordinate safe access to the automatically unmarshalled configuration fields. Remove this functionality, and update documentation to illustrate a safe way to do this. * better Fprint example * fix unit tests * log when OnConfigurationChange fails through OnActivate * clarify lifecycle when OnConfigurationChange returns an error * call SetAPI even if OnConfigurationChange not implemented --- plugin/example_hello_world_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'plugin/example_hello_world_test.go') diff --git a/plugin/example_hello_world_test.go b/plugin/example_hello_world_test.go index 0da171eb7..26f25c91a 100644 --- a/plugin/example_hello_world_test.go +++ b/plugin/example_hello_world_test.go @@ -12,7 +12,7 @@ type HelloWorldPlugin struct { } func (p *HelloWorldPlugin) ServeHTTP(c *plugin.Context, w http.ResponseWriter, r *http.Request) { - fmt.Fprintf(w, "Hello, world!") + fmt.Fprint(w, "Hello, world!") } // This example demonstrates a plugin that handles HTTP requests which respond by greeting the -- cgit v1.2.3-1-g7c22