Skip to content

Commit 37c737e

Browse files
committed
hide socket property in AMQPClient
1 parent e929514 commit 37c737e

11 files changed

Lines changed: 476 additions & 467 deletions

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
premaster.txt
12
package-lock.json
23
# Logs
34
logs

amqpclient.mjs

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
import AMQPBaseClient from './amqpclient-base.mjs'
2-
import AMQPView from './amqpview.mjs'
1+
import AMQPBaseClient from './lib/amqpclient-base.mjs'
2+
import AMQPError from './lib/amqp-error.mjs'
3+
import AMQPView from './lib/amqp-view.mjs'
34
import { Buffer } from 'buffer'
45
import net from 'net'
56
import tls from 'tls'
@@ -18,11 +19,15 @@ export default class AMQPClient extends AMQPBaseClient {
1819
}
1920

2021
connect() {
21-
this.socket = this.tls ? this.connectTLS() : this.connectPlain()
22+
const socket = this.tls ? this.connectTLS() : this.connectPlain()
23+
Object.defineProperty(this, 'socket', {
24+
value: socket,
25+
enumerable: false // hide it from console.log etc.
26+
})
2227
return new Promise((resolve, reject) => {
2328
this.resolvePromise = resolve
2429
this.rejectPromise = reject
25-
this.socket.on('error', reject)
30+
this.socket.on('error', (err) => reject(new AMQPError(err, this)))
2631
})
2732
}
2833

0 commit comments

Comments
 (0)