CASImpl:grantServiceTicket() Javadocs are misleading

Description

While looking at CentralAuthenticationServiceImpl.java, I noticed something strange. There are two overloaded methods for grantServiceTicket().

  • The one that accepts ‘credentials’ says in its javadocs: ‘throws IllegalArgumentException if TicketGrantingTicket ID, Credentials or Service are null.’ [1]

  • The one that does not accept ‘credentials’ passed null instead. No exceptions are thrown. [2]


I am guessing the docs are misleading.

Environment

None

Status

Assignee

ScottS

Reporter

Misagh Moayyed

Labels

None

Estimated End Date

None

Audience

None

Components

Fix versions

Affects versions

3.5.1
3.5.0

Priority

Trivial