server: fix 500 for unauthed get of shots page without shots

e.g. from a private window

server/src/pages/shotindex/model.js

@@ -28,7 +28,7 @@ exports.createModel = function(req) {
       image.url = createProxyUrl(req, image.url);
     }
   }
-  if (shots !== null) {
+  if (shots !== null && shots !== undefined) {
     shots = shots.map((shot) => ({id: shot.id, json: shot.asRecallJson()}));
   }
   let jsonModel = Object.assign(

server/src/pages/shotindex/view.js

@@ -52,8 +52,10 @@ class Body extends React.Component {
       return this.renderShotsLoading();
     }
     let children = [];
-    for (let shot of this.props.shots) {
-      children.push(<Card shot={shot} downloadUrl={this.props.downloadUrls[shot.id]} abTests={this.props.abTests} clipUrl={shot.urlDisplay} isOwner={this.props.isOwner} staticLink={this.props.staticLink} isExtInstalled={this.props.isExtInstalled} key={shot.id} />);
+    if (this.props.shots && this.props.shots.length) {
+      for (let shot of this.props.shots) {
+        children.push(<Card shot={shot} downloadUrl={this.props.downloadUrls[shot.id]} abTests={this.props.abTests} clipUrl={shot.urlDisplay} isOwner={this.props.isOwner} staticLink={this.props.staticLink} isExtInstalled={this.props.isExtInstalled} key={shot.id} />);
+      }
     }
 
     if (children.length === 0) {

test/server/test_responses.py

@@ -115,10 +115,10 @@ def test_my_shots_page():
 
         shot_page = user.read_my_shots()
 
+    # e.g. direct navigation to /shots in private window
     unauthed_user = ScreenshotsClient()
     response = unauthed_user.get_uri("/shots")
-    if response.status_code != 500:  # TODO: fix direct navigation to /shots in private window
-        response.raise_for_status()
+    response.raise_for_status()
 
 
 if __name__ == "__main__":