Return errors only HSP-9244

cmd/spectral.go

@@ -1,7 +1,7 @@
 package main
 
 import (
-	"io/ioutil"
+	"errors"
 	"log"
 	"os"
 	"os/exec"
@@ -33,10 +33,11 @@ var spectral = Spectral{
 }
 
 type SpectralLintOpts struct {
-	Ruleset   string
-	FilePath  string
-	Format    string
-	SkipRules []string
+	Ruleset             string
+	FilePath            string
+	Format              string
+	SkipRules           []string
+	DisplayOnlyFailures bool
 }
 
 func (opts *SpectralLintOpts) Output() string {
@@ -45,6 +46,9 @@ func (opts *SpectralLintOpts) Output() string {
 
 func (opts *SpectralLintOpts) ToArgs() []string {
 	args := []string{"lint", "--quiet", "--ruleset", opts.Ruleset, "--format", opts.Format, "--output", opts.Output()}
+	if opts.DisplayOnlyFailures {
+		args = append(args, "--display-only-failures")
+	}
 	for _, skipRule := range opts.SkipRules {
 		args = append(args, "--skip-rule", skipRule)
 	}
@@ -57,7 +61,8 @@ func (spectral *Spectral) Lint(opts SpectralLintOpts) (string, error) {
 	cmd := exec.Command(spectral.Path, opts.ToArgs()...)
 	log.Println("running command:", cmd.Args)
 	stdoutBytes, err := cmd.Output()
-	exitErr, isExitErr := err.(*exec.ExitError)
+	var exitErr *exec.ExitError
+	isExitErr := errors.As(err, &exitErr)
 	if err != nil {
 		if isExitErr && exitErr.ProcessState.ExitCode() == 1 {
 			log.Println("There seem to be critical linting errors!")
@@ -67,7 +72,7 @@ func (spectral *Spectral) Lint(opts SpectralLintOpts) (string, error) {
 		}
 	}
 
-	outputBytes, err := ioutil.ReadFile(opts.Output())
+	outputBytes, err := os.ReadFile(opts.Output())
 	if err != nil {
 		log.Println("Failed to open output file:", opts.Output())
 		return "", err

cmd/valigator.go

@@ -13,18 +13,20 @@ import (
 )
 
 type ValigatorConfig struct {
-	Host      string   `json:"host"`
-	Port      int      `json:"port"`
-	BasePath  string   `json:"basePath"`
-	SkipRules []string `json:"skipRules"`
+	Host                string   `json:"host"`
+	Port                int      `json:"port"`
+	BasePath            string   `json:"basePath"`
+	DisplayOnlyFailures bool     `json:"displayOnlyFailures"`
+	SkipRules           []string `json:"skipRules"`
 }
 
 func NewValigatorConfig(configFile string) *ValigatorConfig {
 	config := ValigatorConfig{
-		Host:      "0.0.0.0",
-		Port:      8081,
-		BasePath:  "/valigator",
-		SkipRules: []string{},
+		Host:                "0.0.0.0",
+		Port:                8081,
+		BasePath:            "/valigator",
+		DisplayOnlyFailures: true,
+		SkipRules:           []string{},
 	}
 
 	bytes, err := os.ReadFile(configFile)
@@ -152,10 +154,11 @@ func (context *ValigatorContext) validate(w http.ResponseWriter, r *http.Request
 	}
 
 	spectralLintOpts := SpectralLintOpts{
-		FilePath:  filePath,
-		Ruleset:   ruleset,
-		Format:    spectralMediaType,
-		SkipRules: context.Config.SkipRules,
+		FilePath:            filePath,
+		Ruleset:             ruleset,
+		Format:              spectralMediaType,
+		DisplayOnlyFailures: context.Config.DisplayOnlyFailures,
+		SkipRules:           context.Config.SkipRules,
 	}
 
 	spectralLintOutput, err := spectral.Lint(spectralLintOpts)
@@ -176,7 +179,11 @@ func (context *ValigatorContext) validate(w http.ResponseWriter, r *http.Request
 	} else {
 		w.Header().Add("Content-Type", "text/plain")
 	}
-
+	isLocalRequest := strings.Contains(r.Host, "localhost")
+	if isLocalRequest {
+		log.Printf("host is %s. Set Access-Control-Allow-Origin: *", r.Host)
+		w.Header().Add("Access-Control-Allow-Origin", "*")
+	}
 	_, err = w.Write([]byte(spectralLintOutput))
 	if err != nil {
 		log.Println("Write spectral lint output to response failed!")

valigator.json

@@ -3,5 +3,6 @@
   "port": 8081,
   "basePath": "/valigator",
   "skipRules": [],
+  "displayOnlyFailures": true,
   "ruleSets": [ "v5", "v10" ]
-}
+}