Merge pull request #5 from brokenhandsio/swift4

Swift 4 support

Author: 0xTim

.travis.yml

@@ -5,7 +5,7 @@ language: generic
 sudo: required
 dist: trusty
 
-osx_image: xcode8.3
+osx_image: xcode9
 before_install:
     - if [ $TRAVIS_OS_NAME == "osx" ]; then
             brew tap vapor/tap;
@@ -23,7 +23,7 @@ script:
     - swift test
 
 after_success:
-    - eval "$(curl -sL https://raw.githubusercontent.com/vapor-community/swift/master/codecov)"
+    - eval "$(curl -sL https://raw.githubusercontent.com/vapor-community/swift/swift-4-codecov/codecov-swift4)"
 
 notifications:
   email:

Package@swift-4.swift

@@ -0,0 +1,18 @@
+// swift-tools-version:4.0
+
+import PackageDescription
+
+let package = Package(
+    name: "VaporOAuth",
+    products: [
+        .library(name: "VaporOAuth", targets: ["VaporOAuth"]),
+    ],
+    dependencies: [
+    	.package(url: "https://github.com/vapor/vapor.git", .upToNextMajor(from: "2.2.0")),
+        .package(url: "https://github.com/vapor/auth-provider.git", .upToNextMajor(from: "1.2.0")),
+    ],
+    targets: [
+        .target(name: "VaporOAuth", dependencies: ["Vapor", "AuthProvider"]),
+        .testTarget(name: "VaporOAuthTests", dependencies: ["VaporOAuth"]),
+    ]
+)

README.md

@@ -3,7 +3,7 @@
     <br>
     <br>
     <a href="https://swift.org">
-        <img src="http://img.shields.io/badge/Swift-3.1-brightgreen.svg" alt="Language">
+        <img src="http://img.shields.io/badge/Swift-4-brightgreen.svg" alt="Language">
     </a>
     <a href="https://travis-ci.org/brokenhandsio/vapor-oauth">
         <img src="https://travis-ci.org/brokenhandsio/vapor-oauth.svg?branch=master" alt="Build Status">
@@ -40,7 +40,7 @@ Vapor OAuth can be added to your Vapor add with a simple provider. To get starte
 ```swift
 dependencies: [
     ...,
-    .Package(url: "https://github.com/brokenhandsio/vapor-oauth", majorVersion: 0)
+    .package(url: "https://github.com/brokenhandsio/vapor-oauth", from: "0.6.0"))
 ]
 ```
 

Sources/VaporOAuth/Helper/Helper.swift

@@ -60,7 +60,11 @@ extension Request {
             throw Abort(.forbidden)
         }
 
+        #if swift(>=4)
+        let token  = String(authHeader[authHeader.index(authHeader.startIndex, offsetBy: 7)...])
+        #else
         let token = authHeader.substring(from: authHeader.index(authHeader.startIndex, offsetBy: 7))
+        #endif
 
         guard !token.isEmpty else {
             throw Abort(.forbidden)

Tests/VaporOAuthTests/AuthorizationTests/AuthorizationRequestTests.swift

@@ -57,7 +57,7 @@ class AuthorizationRequestTests: XCTestCase {
         #if os(macOS) || os(iOS) || os(tvOS) || os(watchOS)
             let thisClass = type(of: self)
             let linuxCount = thisClass.allTests.count
-            let darwinCount = Int(thisClass.defaultTestSuite().testCaseCount)
+            let darwinCount = Int(thisClass.defaultTestSuite.testCaseCount)
             XCTAssertEqual(linuxCount, darwinCount, "\(darwinCount - linuxCount) tests are missing from allTests")
         #endif
     }

Tests/VaporOAuthTests/AuthorizationTests/AuthorizationResponseTests.swift

@@ -72,7 +72,7 @@ class AuthorizationResponseTests: XCTestCase {
         #if os(macOS) || os(iOS) || os(tvOS) || os(watchOS)
             let thisClass = type(of: self)
             let linuxCount = thisClass.allTests.count
-            let darwinCount = Int(thisClass.defaultTestSuite().testCaseCount)
+            let darwinCount = Int(thisClass.defaultTestSuite.testCaseCount)
             XCTAssertEqual(linuxCount, darwinCount, "\(darwinCount - linuxCount) tests are missing from allTests")
         #endif
     }

Tests/VaporOAuthTests/DefaultImplementationTests/DefaultImplementationTests.swift

@@ -34,7 +34,7 @@ class DefaultImplementationTests: XCTestCase {
         #if os(macOS) || os(iOS) || os(tvOS) || os(watchOS)
             let thisClass = type(of: self)
             let linuxCount = thisClass.allTests.count
-            let darwinCount = Int(thisClass.defaultTestSuite().testCaseCount)
+            let darwinCount = Int(thisClass.defaultTestSuite.testCaseCount)
             XCTAssertEqual(linuxCount, darwinCount, "\(darwinCount - linuxCount) tests are missing from allTests")
         #endif
     }

Tests/VaporOAuthTests/GrantTests/AuthorizationCodeTokenTests.swift

@@ -66,7 +66,7 @@ class AuthorizationCodeTokenTests: XCTestCase {
         #if os(macOS) || os(iOS) || os(tvOS) || os(watchOS)
             let thisClass = type(of: self)
             let linuxCount = thisClass.allTests.count
-            let darwinCount = Int(thisClass.defaultTestSuite().testCaseCount)
+            let darwinCount = Int(thisClass.defaultTestSuite.testCaseCount)
             XCTAssertEqual(linuxCount, darwinCount, "\(darwinCount - linuxCount) tests are missing from allTests")
         #endif
     }

Tests/VaporOAuthTests/GrantTests/ClientCredentialsTokenTests.swift

@@ -61,7 +61,7 @@ class ClientCredentialsTokenTests: XCTestCase {
         #if os(macOS) || os(iOS) || os(tvOS) || os(watchOS)
             let thisClass = type(of: self)
             let linuxCount = thisClass.allTests.count
-            let darwinCount = Int(thisClass.defaultTestSuite().testCaseCount)
+            let darwinCount = Int(thisClass.defaultTestSuite.testCaseCount)
             XCTAssertEqual(linuxCount, darwinCount, "\(darwinCount - linuxCount) tests are missing from allTests")
         #endif
     }

Tests/VaporOAuthTests/GrantTests/ImplicitGrantTests.swift

@@ -88,7 +88,7 @@ class ImplicitGrantTests: XCTestCase {
         #if os(macOS) || os(iOS) || os(tvOS) || os(watchOS)
             let thisClass = type(of: self)
             let linuxCount = thisClass.allTests.count
-            let darwinCount = Int(thisClass.defaultTestSuite().testCaseCount)
+            let darwinCount = Int(thisClass.defaultTestSuite.testCaseCount)
             XCTAssertEqual(linuxCount, darwinCount, "\(darwinCount - linuxCount) tests are missing from allTests")
         #endif
     }