Refactor deprecated code. Fix linter.

plugin.go

@@ -77,7 +77,7 @@ web stop`,
 			}
 
 			if url, _ := getExistingWeb(webPidFile, webRunFlags.PluginDir); url != "" {
-				return fmt.Errorf("Another web UI is already running at %s\nPlease stop the existing server before starting a new one.", url)
+				return fmt.Errorf("another web UI is already running at %s\nPlease stop the existing server before starting a new one", url)
 			}
 
 			if foreground {

server/api.go

@@ -122,7 +122,7 @@ func (l *launchrServer) basePath() string {
 }
 
 func (l *launchrServer) GetActions(w http.ResponseWriter, _ *http.Request) {
-	actions := l.actionMngr.AllRef()
+	actions := l.actionMngr.All()
 	var result = make([]ActionShort, 0, len(actions))
 	for _, a := range actions {
 		ab, err := apiActionShort(a)
@@ -142,7 +142,7 @@ func (l *launchrServer) GetActions(w http.ResponseWriter, _ *http.Request) {
 
 func (l *launchrServer) GetActionByID(w http.ResponseWriter, _ *http.Request, id string) {
 	// @todo return executing actions for a show page.
-	a, ok := l.actionMngr.GetRef(id)
+	a, ok := l.actionMngr.Get(id)
 	if !ok {
 		sendError(w, http.StatusNotFound, fmt.Sprintf("action with id %q is not found", id))
 		return

server/server.go

@@ -44,6 +44,7 @@ type RunOptions struct {
 	ProxyClient string
 }
 
+// BaseURL returns base url for run options.
 func (o RunOptions) BaseURL() string {
 	return "http://localhost:" + strings.Split(o.Addr, ":")[1]
 }

web_runner.go

@@ -139,7 +139,7 @@ func stopWeb(pidFile, pluginDir string) (err error) {
 	}
 
 	if checkHealth(serverRunInfo.URL) {
-		return fmt.Errorf("The web UI is currently running at %s\nPlease stop it through the user interface or terminate the process.", serverRunInfo.URL)
+		return fmt.Errorf("the web UI is currently running at %s\nPlease stop it through the user interface or terminate the process", serverRunInfo.URL)
 	}
 
 	launchr.Term().Success().Println(onSuccess)
@@ -226,7 +226,7 @@ func cleanupPluginTemp(dir string) {
 
 // checkHealth helper to check if server is available by request.
 func checkHealth(url string) bool {
-	resp, err := http.Head(url)
+	resp, err := http.Head(url) //nolint G107 // @todo URL may come from user input, potential vulnerability.
 	if err != nil {
 		// Error is thrown on an incorrect url.
 		panic(err)