Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,16 @@ bin
dist
build
doc/apidoc
release.properties
target
.DS_Store
.settings
.classpath
.project
.flattened-pom.xml

*.releaseBackup

# IntelliJ IDEA
*.iml
.idea/*
17 changes: 17 additions & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,23 @@
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-javadoc-plugin</artifactId>
<version>3.3.1</version>
<executions>
<execution>
<id>attach-javadocs</id>
<goals>
<goal>jar</goal>
</goals>
<configuration>
<failOnWarnings>false</failOnWarnings>
<failOnError>false</failOnError>
</configuration>
</execution>
</executions>
</plugin>
</plugins>
</build>

Expand Down
8 changes: 4 additions & 4 deletions src/main/java/org/tinyradius/attribute/RadiusAttribute.java
Original file line number Diff line number Diff line change
Expand Up @@ -96,8 +96,6 @@ public void setAttributeValue(String value) {
* Gets the value of this attribute as a string.
*
* @return value
* @exception RadiusException
* if the value is invalid
*/
public String getAttributeValue() {
return RadiusUtil.getHexString(getAttributeData());
Expand Down Expand Up @@ -167,8 +165,10 @@ public byte[] writeAttribute() {

/**
* Reads in this attribute from the passed byte array.
*
* @param data
* @param data data buffer
* @param offset the offset to read
* @param length the amount of data to read
* @throws RadiusException when length is less than 2
*/
public void readAttribute(byte[] data, int offset, int length) throws RadiusException {
if (length < 2)
Expand Down
2 changes: 1 addition & 1 deletion src/main/java/org/tinyradius/packet/CoaRequest.java
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

/**
* CoA packet. Thanks to Michael Krastev.
* @author Michael Krastev <mkrastev@gmail.com>
* @author Michael Krastev
*/
public class CoaRequest extends RadiusPacket {

Expand Down
5 changes: 2 additions & 3 deletions src/main/java/org/tinyradius/proxy/RadiusProxy.java
Original file line number Diff line number Diff line change
Expand Up @@ -105,8 +105,7 @@ public void setProxyPort(int proxyPort) {
* Sets the socket timeout.
*
* @param socketTimeout
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EmptyBlockTag: A block tag (@param, @return, @throws, @deprecated) has an empty description. Block tags without descriptions don't add much value for future readers of the code; consider removing the tag entirely or adding a description. (details)
(at-me in a reply with help or ignore)

* socket timeout, >0 ms
* @throws SocketException
* @throws SocketException when socket timeout, >0 ms
*/
public void setSocketTimeout(int socketTimeout) throws SocketException {
super.setSocketTimeout(socketTimeout);
Expand Down Expand Up @@ -214,7 +213,7 @@ protected void proxyPacketReceived(RadiusPacket packet, InetSocketAddress remote
*
* @param packet
* the packet to proxy
* @param proxyCon
* @param proxyConnection
* the RadiusProxyConnection for this packet
* @throws IOException
*/
Expand Down
3 changes: 2 additions & 1 deletion src/main/java/org/tinyradius/test/TestClient.java
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@ public class TestClient {

/**
* Radius command line client.
* <br/>Usage: TestClient <i>hostName sharedSecret userName password</i>
*
* Usage: TestClient <i>hostName sharedSecret userName password</i>
* @param args arguments
* @throws Exception
*/
Expand Down
2 changes: 1 addition & 1 deletion src/main/java/org/tinyradius/util/RadiusClient.java
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,7 @@ public int getRetryCount() {
* Sets the retry count for failed transmissions.
*
* @param retryCount
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EmptyBlockTag: A block tag (@param, @return, @throws, @deprecated) has an empty description. Block tags without descriptions don't add much value for future readers of the code; consider removing the tag entirely or adding a description. (details)
(at-me in a reply with help or ignore)

* retry count, >0
* @throws IllegalArgumentException when retry count is not positive
*/
public void setRetryCount(int retryCount) {
if (retryCount < 1)
Expand Down
12 changes: 4 additions & 8 deletions src/main/java/org/tinyradius/util/RadiusServer.java
Original file line number Diff line number Diff line change
Expand Up @@ -224,13 +224,12 @@ public int getSocketTimeout() {
/**
* Sets the socket timeout.
*
* @param socketTimeout
* socket timeout, >0 ms
* @throws SocketException
* @param socketTimeout when socket timeout, >0 ms
* @throws SocketException when socketTime is not positive
*/
public void setSocketTimeout(int socketTimeout) throws SocketException {
if (socketTimeout < 1)
throw new IllegalArgumentException("socket tiemout must be positive");
throw new IllegalArgumentException("socket timeout must be positive");
this.socketTimeout = socketTimeout;
if (authSocket != null)
authSocket.setSoTimeout(socketTimeout);
Expand Down Expand Up @@ -279,7 +278,7 @@ public long getDuplicateInterval() {
* same address.
*
* @param duplicateInterval
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EmptyBlockTag: A block tag (@param, @return, @throws, @deprecated) has an empty description. Block tags without descriptions don't add much value for future readers of the code; consider removing the tag entirely or adding a description. (details)
(at-me in a reply with help or ignore)

* duplicate interval (ms), >0
* @throws IllegalArgumentException when duplicateInterval (ms), >0
*/
public void setDuplicateInterval(long duplicateInterval) {
if (duplicateInterval <= 0)
Expand Down Expand Up @@ -341,8 +340,6 @@ protected void copyProxyState(RadiusPacket request, RadiusPacket answer) {
* Returns when stop() is called.
*
* @throws SocketException
* @throws InterruptedException
*
*/
protected void listenAuth() throws SocketException {
listen(getAuthSocket());
Expand All @@ -353,7 +350,6 @@ protected void listenAuth() throws SocketException {
* Returns when stop() is called.
*
* @throws SocketException
* @throws InterruptedException
*/
protected void listenAcct() throws SocketException {
listen(getAcctSocket());
Expand Down