🐛 fix: Client rejection on auth

README.md

@@ -50,7 +50,7 @@ const server = createServer({
 
 // For development purposes
 server.on('data', (data) => {
-  // Log incoming requests
+  // Log incoming data
   console.log(data)
 })
 
@@ -210,7 +210,7 @@ you can easily implement it in a matter of seconds.
      )
    )
 
-   socket.once('data', (data) => {
+   socket.on('data', (data) => {
      console.log(data)
    })
    ```
@@ -237,7 +237,7 @@ you can easily implement it in a matter of seconds.
      )
    )
 
-   socket1.once('data', (data) => {
+   socket1.on('data', (data) => {
      console.log(data)
    })
 
@@ -252,7 +252,7 @@ you can easily implement it in a matter of seconds.
      )
    )
 
-   socket2.once('data', (data) => {
+   socket2.on('data', (data) => {
      console.log(data)
    })
    ```
@@ -293,7 +293,7 @@ as an argument (userId) to the request handler (connect | bind | associate).
      )
    )
 
-   socket.once('data', (data) => {
+   socket.on('data', (data) => {
      console.log(data)
    })
    ```
@@ -322,7 +322,7 @@ as an argument (userId) to the request handler (connect | bind | associate).
      )
    )
 
-   socket.once('data', (data) => {
+   socket.on('data', (data) => {
      console.log(data)
    })
    ```

src/client/auth/methods/userPass.ts

@@ -49,6 +49,9 @@ class UserPassReqState extends State {
    */
   reply(): void {
     if (this.status !== 0x00) {
+      if (this.context.reject) {
+        this.context.reject('Wrong credentials supplied')
+      }
       this.context.close()
     } else {
       this.context.transitionTo(new RequestState(this.context))

src/client/state/socks5.ts

@@ -30,13 +30,17 @@ export class MethodSelectionState extends State {
    * @returns void
    */
   reply(): void {
-    this.context.handlers.auth
-      .find((method) => {
-        if (this.method && method) {
-          return method.method === this.method.readInt8()
-        }
-      })
-      ?.authenticate(this.context)
+    if (this.method?.readInt8() === -1 && this.context.reject) {
+      this.context.reject('Authentication methods are not acceptable')
+    } else {
+      this.context.handlers.auth
+        .find((method) => {
+          if (this.method && method) {
+            return method.method === this.method.readInt8()
+          }
+        })
+        ?.authenticate(this.context)
+    }
   }
 }