use application-specific CONNECTION_CLOSE for application errors

internal/qerr/quic_error.go

@@ -39,6 +39,7 @@ func CryptoError(tlsAlert uint8, errorMessage string) *QuicError {
 	}
 }
 
+// ApplicationError creates a new QuicError instance for an application error
 func ApplicationError(errorCode ErrorCode, errorMessage string) *QuicError {
 	return &QuicError{
 		ErrorCode:          errorCode,
@@ -65,6 +66,11 @@ func (e *QuicError) IsCryptoError() bool {
 	return e.ErrorCode.isCryptoError()
 }
 
+// IsApplicationError says if this error is an application error
+func (e *QuicError) IsApplicationError() bool {
+	return e.isApplicationError
+}
+
 // Temporary says if the error is temporary.
 func (e *QuicError) Temporary() bool {
 	return false

internal/qerr/quic_error_test.go

@@ -11,6 +11,7 @@ var _ = Describe("QUIC Transport Errors", func() {
 	It("has a string representation", func() {
 		err := Error(FlowControlError, "foobar")
 		Expect(err.Timeout()).To(BeFalse())
+		Expect(err.IsApplicationError()).To(BeFalse())
 		Expect(err.Error()).To(Equal("FLOW_CONTROL_ERROR: foobar"))
 	})
 
@@ -38,13 +39,17 @@ var _ = Describe("QUIC Transport Errors", func() {
 
 		It("says if an error is a crypto error", func() {
 			Expect(Error(FlowControlError, "").IsCryptoError()).To(BeFalse())
-			Expect(CryptoError(42, "").IsCryptoError()).To(BeTrue())
+			err := CryptoError(42, "")
+			Expect(err.IsCryptoError()).To(BeTrue())
+			Expect(err.IsApplicationError()).To(BeFalse())
+
 		})
 	})
 
 	Context("application errors", func() {
 		It("has a string representation for errors with a message", func() {
 			err := ApplicationError(0x42, "foobar")
+			Expect(err.IsApplicationError()).To(BeTrue())
 			Expect(err.Error()).To(Equal("Application error 0x42: foobar"))
 		})