iegomez / mosquitto-go-auth

Auth plugin for mosquitto.
MIT License
495 stars 165 forks source link

Error in run tests with docker and mounted volume #286

Closed alessandroperetti closed 1 year ago

alessandroperetti commented 1 year ago

Hi all,

I spent a lot of time in trying to run tests with a mounted volume in Docker ( avoiding the need to rebuild image according to the documentation) because I got error obtaining VCS status: exit status 128 during the docker run.

I solved it with adding -buildvcs=false in plugin Makefile :

go build -buildvcs=false -buildmode=plugin

Do you have the same problem @iegomez ?

AP

iegomez commented 1 year ago

I did when trying to add test running as Github workflow, that's why I added this a couple of weeks ago: https://github.com/iegomez/mosquitto-go-auth/commit/608a1e6a728f94ec8dd36dfc53e24a65a4e69a4a

alessandroperetti commented 1 year ago

Great! I did not notice it

iegomez commented 1 year ago

It's a Go thing that was introduced some time ago si I'm not very keen to include it as part of the plugin's docs. But at least now we have this issue for people to reference if they face a similar situation.

I'll close this now as it's not really an issue in the plugin.