OpenFn / adaptors

The new home for OpenFn adaptors; re-usable connectors for the most common DPGs and DPI building blocks.
GNU General Public License v3.0
4 stars 8 forks source link

odk: give better error when auth fails #647

Closed josephjclark closed 1 week ago

josephjclark commented 1 week ago

When okd authentication fails, we don't report very robust errors

I also note that if username and password isn't set, we don't even try to auth (I think we assume there's an api token instead).

We should at least raise a warning if authentication is skipped

Taylor was stuck here for ages:

[CLI] ♦ Versions:
         ▸ node.js                 18.17.1
         ▸ cli                     1.4.0
         ▸ @openfn/language-odk    monorepo
[CLI] ✔ Loading adaptors from monorepo at /Users/taylor/build/adaptors
[CLI] ⚠ Skipping auto-install as monorepo is being used
[CLI] ✔ Loaded state from ./tmp/state.json
[CLI] ✔ Compiled all expressions in workflow
do we get here?
[R/T] ✘ Failed step job-1 after 912ms
[R/T] ✘ JobError: Page Not Found
[R/T] ✘ Check state.errors.job-1 for details.
[CLI] ✔ State written to tmp/output.json
[CLI] ⚠ Errors reported in 1 jobs
[CLI] ✔ Finished in 994ms