Adding MIT header to checkstyle checks. (#41)

* Adding checkstyle to check headers.

* Add headers to all src files.

* Adding MIT header to test files.

checkstyle.xml → eng/checkstyle/checkstyle.xml

@@ -12,16 +12,9 @@ what the following rules do, please see the checkstyle configuration
 page at http://checkstyle.sourceforge.net/config.html -->
 
 <module name="Checker">
-  <module name="SuppressionFilter">
-    <property name="file" value="eng/checkstyle-suppressions.xml" />
-  </module>
-
-  <!-- Filter out Checkstyle warnings that have been suppressed with the @SuppressWarnings annotation -->
-  <module name="SuppressWarningsFilter"/>
 
   <module name="FileTabCharacter"/>
 
-
   <module name="NewlineAtEndOfFile"/>
 
   <!-- Checks that a package-info.java file exists for each package. -->
@@ -33,6 +26,12 @@ page at http://checkstyle.sourceforge.net/config.html -->
     <property name="ignorePattern" value="^package.*|^import.*|a href|href|http://|https://|ftp://"/>
   </module>
 
+  <!-- Validate that we have the MIT license at the top of the file -->
+  <module name="Header">
+    <property name="headerFile" value="eng/checkstyle/java.header"/>
+    <property name="fileExtensions" value="java"/>
+  </module>
+
   <!-- All Java AST specific tests live under TreeWalker module. -->
   <module name="TreeWalker">
 

eng/checkstyle/java.header

@@ -0,0 +1,2 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.

pom.xml

@@ -181,10 +181,8 @@
             </dependency>
           </dependencies>
           <configuration>
-            <configLocation>checkstyle.xml</configLocation>
-            <!-- <suppressionsLocation>checkstyle/checkstyle-suppressions.xml</suppressionsLocation> -->
-            <!-- <headerLocation>checkstyle/checkstyle/java.header</headerLocation> -->
-            <propertyExpansion>samedir=</propertyExpansion>
+            <configLocation>eng/checkstyle/checkstyle.xml</configLocation>
+            <suppressionsLocation>eng/checkstyle/checkstyle-suppressions.xml</suppressionsLocation>
             <encoding>UTF-8</encoding>
             <consoleOutput>true</consoleOutput>
             <!-- Remove when https://github.com/Azure/qpid-proton-j-extensions/issues/39 is fixed. -->

src/main/java/com/microsoft/azure/proton/transport/proxy/Proxy.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.proxy;
 

src/main/java/com/microsoft/azure/proton/transport/proxy/ProxyAuthenticationType.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy;
 
 /**

src/main/java/com/microsoft/azure/proton/transport/proxy/ProxyChallengeProcessor.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy;
 
 import java.util.Map;

src/main/java/com/microsoft/azure/proton/transport/proxy/ProxyConfiguration.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy;
 
 import org.slf4j.Logger;

src/main/java/com/microsoft/azure/proton/transport/proxy/ProxyHandler.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.proxy;
 

src/main/java/com/microsoft/azure/proton/transport/proxy/impl/BasicProxyChallengeProcessorImpl.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy.impl;
 
 import com.microsoft.azure.proton.transport.proxy.ProxyChallengeProcessor;

src/main/java/com/microsoft/azure/proton/transport/proxy/impl/Constants.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy.impl;
 
 import java.util.Locale;

src/main/java/com/microsoft/azure/proton/transport/proxy/impl/DigestProxyChallengeProcessorImpl.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy.impl;
 
 import com.microsoft.azure.proton.transport.proxy.ProxyChallengeProcessor;

src/main/java/com/microsoft/azure/proton/transport/proxy/impl/ProxyAuthenticator.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy.impl;
 
 import com.microsoft.azure.proton.transport.proxy.ProxyConfiguration;

src/main/java/com/microsoft/azure/proton/transport/proxy/impl/ProxyHandlerImpl.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.proxy.impl;
 

src/main/java/com/microsoft/azure/proton/transport/proxy/impl/ProxyImpl.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.proxy.impl;
 

src/main/java/com/microsoft/azure/proton/transport/proxy/impl/StringUtils.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy.impl;
 
 /**

src/main/java/com/microsoft/azure/proton/transport/proxy/impl/package-info.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 /**
  * Package containg implementation for {@link com.microsoft.azure.proton.transport.proxy.ProxyHandler} and {@link
  * com.microsoft.azure.proton.transport.proxy.Proxy}.

src/main/java/com/microsoft/azure/proton/transport/proxy/package-info.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 /**
  * Package containing classes for connecting via an HTTP proxy.
  */

src/main/java/com/microsoft/azure/proton/transport/ws/WebSocket.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.ws;
 

src/main/java/com/microsoft/azure/proton/transport/ws/WebSocketHandler.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.ws;
 

src/main/java/com/microsoft/azure/proton/transport/ws/WebSocketHeader.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.ws;
 

src/main/java/com/microsoft/azure/proton/transport/ws/impl/Base64.java

@@ -1,5 +1,5 @@
-// Copyright (c) Microsoft. All rights reserved.
-// Licensed under the MIT license. See LICENSE file in the project root for full license information.
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.ws.impl;
 

src/main/java/com/microsoft/azure/proton/transport/ws/impl/WebSocketHandlerImpl.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.ws.impl;
 

src/main/java/com/microsoft/azure/proton/transport/ws/impl/WebSocketImpl.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.ws.impl;
 

src/main/java/com/microsoft/azure/proton/transport/ws/impl/WebSocketSniffer.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.ws.impl;
 

src/main/java/com/microsoft/azure/proton/transport/ws/impl/WebSocketUpgrade.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.ws.impl;
 

src/main/java/com/microsoft/azure/proton/transport/ws/impl/package-info.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 /**
  * Package containg implementation for {@link com.microsoft.azure.proton.transport.ws.WebSocketHandler} and {@link
  * com.microsoft.azure.proton.transport.ws.WebSocket}.

src/main/java/com/microsoft/azure/proton/transport/ws/package-info.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 /**
  * Package containing classes for connecting via WebSockets.
  */

src/test/java/com/microsoft/azure/proton/transport/proxy/impl/BasicProxyChallengeProcessorImplTest.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy.impl;
 
 import com.microsoft.azure.proton.transport.proxy.ProxyAuthenticationType;
@@ -7,9 +10,18 @@
 import org.junit.Test;
 
 import java.io.IOException;
-import java.net.*;
+import java.net.Authenticator;
+import java.net.InetSocketAddress;
+import java.net.PasswordAuthentication;
+import java.net.Proxy;
+import java.net.ProxySelector;
+import java.net.SocketAddress;
+import java.net.URI;
 import java.nio.charset.StandardCharsets;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.Base64;
+import java.util.List;
+import java.util.Map;
 
 public class BasicProxyChallengeProcessorImplTest {
     private static final String HOSTNAME = "127.0.0.1";

src/test/java/com/microsoft/azure/proton/transport/proxy/impl/DigestProxyChallengeProcessorImplTest.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy.impl;
 
 import com.microsoft.azure.proton.transport.proxy.ProxyAuthenticationType;

src/test/java/com/microsoft/azure/proton/transport/proxy/impl/ProxyAuthenticatorTests.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy.impl;
 
 import com.microsoft.azure.proton.transport.proxy.ProxyAuthenticationType;

src/test/java/com/microsoft/azure/proton/transport/proxy/impl/ProxyConfigurationTest.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy.impl;
 
 import com.microsoft.azure.proton.transport.proxy.ProxyAuthenticationType;

src/test/java/com/microsoft/azure/proton/transport/proxy/impl/ProxyHandlerImplTest.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.proxy.impl;
 

src/test/java/com/microsoft/azure/proton/transport/proxy/impl/ProxyImplTest.java

@@ -1,7 +1,5 @@
-/*
- * Copyright (c) Microsoft. All rights reserved.
- * Licensed under the MIT license. See LICENSE file in the project root for full license information.
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.proxy.impl;
 

src/test/java/com/microsoft/azure/proton/transport/proxy/impl/TestAuthenticator.java

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
 package com.microsoft.azure.proton.transport.proxy.impl;
 
 import java.net.Authenticator;

src/test/java/com/microsoft/azure/proton/transport/ws/impl/Base64Test.java

@@ -1,5 +1,5 @@
-// Copyright (c) Microsoft. All rights reserved.
-// Licensed under the MIT license. See LICENSE file in the project root for full license information.
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.ws.impl;
 

src/test/java/com/microsoft/azure/proton/transport/ws/impl/WebSocketHandlerImplTest.java

@@ -1,23 +1,5 @@
-/*
- *
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements.  See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership.  The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License.  You may obtain a copy of the License at
- *
- *   http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
- *
- */
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
 
 package com.microsoft.azure.proton.transport.ws.impl;
 
@@ -27,10 +9,20 @@
 
 import java.nio.ByteBuffer;
 import java.security.SecureRandom;
-import java.util.*;
-
-import static org.junit.Assert.*;
-import static org.mockito.Mockito.*;
+import java.util.Arrays;
+import java.util.HashMap;
+import java.util.Map;
+import java.util.Random;
+import java.util.Scanner;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+import static org.mockito.Mockito.doReturn;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.spy;
+import static org.mockito.Mockito.times;
+import static org.mockito.Mockito.verify;
 
 public class WebSocketHandlerImplTest {
     @Test