Skip to content

Commit

Permalink
server name not needed now that we use user (#626)
Browse files Browse the repository at this point in the history
  • Loading branch information
yanokwa authored Oct 4, 2022
1 parent 82ee0f0 commit 851f30f
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions lib/worker/worker.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,13 @@ const { sql } = require('slonik');
const { timebound, resolve } = require('../util/promise');
const defaultJobMap = require('./jobs').jobs;

const hostname = require('os').hostname();

// TODO: domain catch/restart? << investigated and seems unlikely.

// we'd love to report to sentry, but it's much more important that we don't fail
// in our error handling code. so we will do anything to pass this and move on.
const reporter = (Sentry) => (err) => {
/* eslint-disable */ // i don't like anything it suggests here.
try { Sentry.captureException(err, { server_name: hostname }); }
try { Sentry.captureException(err); }
catch (ierr) {
try { process.stderr.write(inspect(err) + '\n'); process.stderr.write(inspect(ierr) + '\n'); }
catch (_) { /* too scared to try anything at this point */ }
Expand Down

0 comments on commit 851f30f

Please sign in to comment.