[ 
https://issues.apache.org/jira/browse/CASSANDRA-13662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16152525#comment-16152525
 ] 

Stefan Podkowinski commented on CASSANDRA-13662:
------------------------------------------------

Thanks for taking a look at this. When it comes to unit testing, we could 
create a test to verify the correct opcode > UnsupportedMessageCodec mapping, 
which seems to be kinda silly. Same with writing a unit test for 
UnsupportedMessageCodec, which always just throws an exception for both 
methods. Writing a integration test would require to make the driver send 
CREDENTIAL, which I think would be non-trivial for this type of low-level 
operation, but I'm not a driver expert. 

> Remove unsupported CREDENTIALS message
> --------------------------------------
>
>                 Key: CASSANDRA-13662
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-13662
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Auth, CQL
>            Reporter: Stefan Podkowinski
>            Assignee: Stefan Podkowinski
>            Priority: Minor
>              Labels: security
>             Fix For: 4.x
>
>
> Remove CREDENTIAL message, as protocol v1 isn't supported anyways. Let's try 
> not to keep unused legacy classes around for any security relevant features. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to