reenable header injection

Author: timsuchanek

packages/graphql-playground-react/src/components/Playground.tsx

@@ -83,7 +83,6 @@ export interface Props {
   configPath?: string
   createApolloLink?: (session: Session) => ApolloLink
   workspaceName?: string
-  shouldInjectHeaders: boolean
 }
 
 export interface ReduxProps {
@@ -152,9 +151,7 @@ export class Playground extends React.PureComponent<Props & ReduxProps, State> {
     // init redux
     this.props.initState(getWorkspaceId(this.props), this.props.endpoint)
     this.props.setConfigString(this.props.configString)
-    if (this.props.shouldInjectHeaders) {
-      this.props.injectHeaders(this.props.headers, this.props.endpoint)
-    }
+    this.props.injectHeaders(this.props.headers, this.props.endpoint)
   }
 
   componentDidMount() {
@@ -193,10 +190,7 @@ export class Playground extends React.PureComponent<Props & ReduxProps, State> {
     if (this.props.subscriptionEndpoint !== nextProps.subscriptionEndpoint) {
       setSubscriptionEndpoint(nextProps.subscriptionEndpoint)
     }
-    if (
-      nextProps.headers !== this.props.headers &&
-      this.props.shouldInjectHeaders
-    ) {
+    if (nextProps.headers !== this.props.headers) {
       this.props.injectHeaders(nextProps.headers, nextProps.endpoint)
     }
     if (nextProps.configString !== this.props.configString) {

packages/graphql-playground-react/src/components/PlaygroundWrapper.tsx

@@ -64,7 +64,6 @@ export interface State {
   activeProjectName?: string
   activeEnv?: string
   headers?: any
-  shouldInjectHeaders: boolean
 }
 
 class PlaygroundWrapper extends React.Component<
@@ -119,7 +118,6 @@ class PlaygroundWrapper extends React.Component<
       activeEnv,
       activeProjectName: projectName,
       headers,
-      shouldInjectHeaders: false,
     }
   }
 
@@ -131,7 +129,6 @@ class PlaygroundWrapper extends React.Component<
     try {
       const headers = getParameterByName('headers', endpoint)
       if (headers) {
-        this.setState({ shouldInjectHeaders: true })
         return { headers: JSON.parse(headers), endpoint: splitted[0] }
       }
     } catch (e) {
@@ -325,7 +322,6 @@ class PlaygroundWrapper extends React.Component<
                 headers={this.state.headers}
                 configPath={this.props.configPath}
                 workspaceName={this.state.activeProjectName}
-                shouldInjectHeaders={this.state.shouldInjectHeaders}
               />
             </App>
           </OldThemeProvider>

packages/graphql-playground-react/src/localDevIndex.tsx

@@ -54,7 +54,7 @@ const config = {
             url: 'https://eu1.prisma.sh/lol/asdf/dev',
             headers: {
               Authorization:
-                'Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJkYXRhIjp7InNlcnZpY2UiOiJwcmlzbWEtYWlyYm5iLWV4YW1wbGVAZGV2Iiwicm9sZXMiOlsiYWRtaW4iXX0sImlhdCI6MTUyMjY4NDkzNiwiZXhwIjoxNTIzMjg5NzM2fQ.1wnkIJrAplDOznjEzLj8sQERL6dcAM0zjqxOGQBXGn0',
+                'Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJkYXRhIjp7InNlcnZpY2UiOiJhc2RmQGRldiIsInJvbGVzIjpbImFkbWluIl19LCJpYXQiOjE1MjM1MTg3NTYsImV4cCI6MTUyNDEyMzU1Nn0.fzKhXa1JpN9M1UGTbS6p2KMUWDrKLxYD3i3a9eVfOQQ',
             },
           },
         },

packages/graphql-playground-react/src/state/sessions/reducers.ts

@@ -278,7 +278,10 @@ const reducer = handleActions(
       })
     },
     SET_SCROLL_TOP: (state, { payload: { sessionId, scrollTop } }) => {
-      return state.setIn(['sessions', sessionId, 'scrollTop'], scrollTop)
+      if (state.sessions.get(sessionId)) {
+        return state.setIn(['sessions', sessionId, 'scrollTop'], scrollTop)
+      }
+      return state
     },
     SCHEMA_FETCHING_SUCCESS: (state, { payload }) => {
       const newSessions = state