Cannot read property 'message' of null

[kitsune7]

I set the token to expire after 1440 minutes (24 hours), but if I try to make any requests using the token after that time, I get this error message.

[cheeaun]

@kitsune7 hey do you have a sample code somewhere?

[kitsune7]

Yeah, what parts do you want to see? It's on a private repo, but I can share segments.
I set an authentication token after logging in like so:

const ONE_DAY_IN_MINUTES = 1440
store.set('auth-token', data.auth.token, ONE_DAY_IN_MINUTES)

I use the token in authMiddleware.js like this:

import {applyMiddleware} from 'apollo-client'
import CacheStore from 'react-native-cache-store'
import {isEmpty} from 'lodash'

export default {
  async applyMiddleware(req, next) {

    const {operationName} = req.request

    if (!req.options.headers) {
      req.options.headers = {} // Create the header object if needed.
    }

    const token = await CacheStore.get('auth-token')

	  if (!isEmpty(token)) {
		  req.options.headers.authorization = `Bearer ${token}`
	  }

	  if (operationName === 'auth') {
		  req.options.headers.authorization && delete req.options.headers.authorization
	  }

	  next()
  }
}

I also made it so when my app starts up it will skip the login screen if it still has an authentication token:

componentWillMount = async () => {
    const token = await store.get('auth-token')

    const { currentScene, currentSceneProps } = this.props
    let navigateTo = currentScene
    
    if (!isEmpty(token)) {
      if (!currentScene || currentScene === 'Init') {
        navigateTo = "Dashboard"
      }
      //Actions['Dashboard']({...currentSceneProps, autoLoaded: true})
      Actions[navigateTo]({...currentSceneProps, autoLoaded: true})
    } else {
      Actions.Login()
    }
  }

[nicolashemonic]

Hi guys,

I experience same issue, in my case the error cannot read property of null is thrown by a try catch(e) who receive null instead of an exception who report a e.message.

After investigation this issue is a side effect of the Cache behavior. The Cache do not verify the value before resolving the promise, in the case of a null value the promise is resolve whereas it should not.

Find below my correction:

	get(key: string) {
		const theKey = CACHE_PREFIX + key;
		const exprKey = CACHE_EXPIRATION_PREFIX + key;
		return AsyncStorage.getItem(exprKey).then(expiry => {
			if (expiry && currentTime() >= parseInt(expiry, 10)) {
				AsyncStorage.multiRemove([exprKey, theKey]);
				return Promise.reject(null);
			}
			return AsyncStorage.getItem(theKey).then(item => {

                                // CORRECTION: reject Promise if item is null
				if (item) {
					return Promise.resolve(JSON.parse(item));
				}
				return Promise.reject(null);
			});
		});
	},

And for information, you should not write new Promise.reject() because reject is a static method of Promise class.

I can pull request if you want 😉

[cheeaun]

@nicolashemonic a PR would be very helpful 🙇

[nicolashemonic]

@cheeaun PR done! Let me know if it's good to you 🙂