René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Fingerprinting in the upload-sarif action fails with directories","articleBody":"The Android linter started supporting sarif files in the latest version (4.2). When trying it out in one of my projects (https://github.com/jellyfin/jellyfin-androidtv) it failed uploading the artifact. This is because one of the linter rules adds a directory as uri in the locations object. The fingerprinting code assumes it is a file and tries to fingerprint the directory. The specification says that an uri should be pointing to a file (https://rawgit.com/sarif-standard/sarif-spec/master/Static Analysis Results Interchange Format (SARIF).html#fileformat-uriproperties) but I don't expect this action to fail.\r\n\r\nError:\r\n\r\n```\r\nRun github/codeql-action/upload-sarif@v1\r\n  with:\r\n    sarif_file: .\r\n    checkout_path: /home/runner/work/jellyfin-androidtv/jellyfin-androidtv\r\n    token: ***\r\n    matrix: null\r\n  env:\r\n    JAVA_HOME: /opt/hostedtoolcache/Java_Adopt_jdk/11.0.10-9/x64\r\nUploading sarif files: [\"/home/runner/work/jellyfin-androidtv/jellyfin-androidtv/app/build/reports/detekt/detekt.sarif\",\"/home/runner/work/jellyfin-androidtv/jellyfin-androidtv/app/build/reports/lint-results.sarif\"]\r\nError: EISDIR: illegal operation on a directory, read\r\nError: EISDIR: illegal operation on a directory, read\r\n    at Object.readSync (fs.js:506:3)\r\n    at tryReadSync (fs.js:331:20)\r\n    at Object.readFileSync (fs.js:368:19)\r\n    at Object.addFingerprints (/home/runner/work/_actions/github/codeql-action/v1/lib/fingerprints.js:239:33)\r\n    at uploadFiles (/home/runner/work/_actions/github/codeql-action/v1/lib/upload-lib.js:249:33)\r\n    at Object.uploadFromActions (/home/runner/work/_actions/github/codeql-action/v1/lib/upload-lib.js:123:18)\r\n    at async run (/home/runner/work/_actions/github/codeql-action/v1/lib/upload-sarif-action.js:34:29)\r\n    at async runWrapper (/home/runner/work/_actions/github/codeql-action/v1/lib/upload-sarif-action.js:46:9) {\r\n  errno: -21,\r\n  syscall: 'read',\r\n  code: 'EISDIR'\r\n}\r\n```\r\n\r\nFile that should reproduce the issue:\r\n\r\n[android-lint.sarif.txt](https://github.com/github/codeql-action/files/6429409/android-lint.sarif.txt)","author":{"url":"https://github.com/nielsvanvelzen","@type":"Person","name":"nielsvanvelzen"},"datePublished":"2021-05-05T18:28:24.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/478/codeql-action/issues/478"}