Fixes #125, version bump

Author: rgommezz

README.md

@@ -72,7 +72,7 @@ withNetworkConnectivity(config: Config): (WrappedComponent) => EnhancedComponent
 type Config = {
   withRedux?: boolean = false,
   timeout?: number = 3000,
-  pingServerUrl?: string = 'http://www.google.com/',
+  pingServerUrl?: string = 'https://www.google.com/',
   withExtraHeadRequest?: boolean = true,
   checkConnectionInterval?: number = 0,
   checkIntervalOfflineOnly?: boolean = false,
@@ -86,7 +86,7 @@ type Config = {
 
 `timeout`: amount of time (in ms) that the component should wait for the ping response. Defaults to 3s.
 
-`pingServerUrl`: remote server to ping to. It defaults to http://www.google.com/ since it's probably one the most stable servers out there, but you can provide your own if needed.
+`pingServerUrl`: remote server to ping to. It defaults to https://www.google.com/ since it's probably one the most stable servers out there, but you can provide your own if needed.
 
 `withExtraHeadRequest`: flag that denotes whether the extra ping check will be performed or not. Defaults to `true`.
 
@@ -119,7 +119,7 @@ React component that accepts a function as children. It allows you to decouple y
 type Props = {
   children: (isConnected: boolean) => React$Element<any>
   timeout?: number = 3000,
-  pingServerUrl?: string = 'http://www.google.com/',
+  pingServerUrl?: string = 'https://www.google.com/',
   withExtraHeadRequest?: boolean = true,
 }
 ```
@@ -266,7 +266,7 @@ type Config = {
 This is the setup you need to put in place for libraries such as `redux-saga` or `redux-observable`, which rely on plain actions being dispatched to trigger async flow:
 
 `regexActionType`: regular expression to indicate the action types to be intercepted in offline mode.
-By default it's configured to intercept actions for fetching data following the Redux [convention](http://redux.js.org/docs/advanced/AsyncActions.html). That means that it will intercept actions with types such as `FETCH_USER_ID_REQUEST`, `FETCH_PRODUCTS_REQUEST` etc.
+By default it's configured to intercept actions for fetching data following the Redux [convention](https://redux.js.org/docs/advanced/AsyncActions.html). That means that it will intercept actions with types such as `FETCH_USER_ID_REQUEST`, `FETCH_PRODUCTS_REQUEST` etc.
 
 `actionTypes`: array with additional action types to intercept that don't fulfil the RegExp criteria. For instance, it's useful for actions that carry along refreshing data, such as `REFRESH_LIST`.
 
@@ -419,7 +419,7 @@ fetchData.meta = {
 Utility function that allows you to query for internet connectivity on demand. If you have integrated this library with redux, you can then dispatch a `CONNECTION_CHANGE` action type to inform the `network` reducer accordingly and keep it up to date. Check the example below.
 
 ```js
-checkInternetConnection(timeout?: number = 3000, url?: string = 'http://www.google.com/'): Promise<boolean>
+checkInternetConnection(timeout?: number = 3000, url?: string = 'https://www.google.com/'): Promise<boolean>
 ```
 
 ##### Example

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-offline",
-  "version": "3.13.0",
+  "version": "3.14.0",
   "description": "Handy toolbelt to deal with offline mode in React Native applications. Cross-platform, provides a smooth redux integration.",
   "main": "./src/index.js",
   "author": "Raul Gomez Acuña <raulgdeveloper@gmail.com> (https://github.com/rgommezz)",

src/ConnectivityRenderer.js

@@ -30,7 +30,7 @@ class ConnectivityRenderer extends Component<DefaultProps, Props, State> {
 
   static defaultProps: DefaultProps = {
     timeout: 3000,
-    pingServerUrl: 'http://www.google.com/',
+    pingServerUrl: 'https://www.google.com/',
     withExtraHeadRequest: true,
   };
 

src/checkInternetAccess.js

@@ -4,7 +4,7 @@ import type { HTTPMethod } from './types';
 
 export default function checkInternetAccess(
   timeout: number = 3000,
-  url: string = 'http://www.google.com/',
+  url: string = 'https://www.google.com/',
   method: HTTPMethod = 'HEAD',
 ): Promise<boolean> {
   return new Promise((resolve: (value: boolean) => void) => {

src/checkInternetConnection.js

@@ -13,7 +13,7 @@ import checkInternetAccess from './checkInternetAccess';
  */
 export default function checkInternetConnection(
   timeout: number = 3000,
-  url: string = 'http://www.google.com/',
+  url: string = 'https://www.google.com/',
 ): Promise<boolean> {
   let connectionChecked: Promise<boolean>;
   if (Platform.OS === 'ios') {

src/makeHttpRequest.js

@@ -38,7 +38,8 @@ export default function makeHttpRequest({
 
     xhr.open(method, url);
     xhr.onload = function onLoad() {
-      if (this.status >= 200 && this.status < 300) {
+      // 3xx is a valid response for us, since the server was reachable
+      if (this.status >= 200 && this.status < 400) {
         clearTimeout(tOut);
         resolve(xhr.response);
       } else {

src/withNetworkConnectivity.js

@@ -31,7 +31,7 @@ type State = {
 const withNetworkConnectivity = ({
   withRedux = false,
   timeout = 3000,
-  pingServerUrl = 'http://www.google.com/',
+  pingServerUrl = 'https://www.google.com/',
   withExtraHeadRequest = true,
   checkConnectionInterval = 0,
   checkIntervalOfflineOnly = false,