8c start

Author: mluukkai

src/content/8/en/part8c.md

@@ -11,10 +11,10 @@ We will now add user management to our application, but let's first start using
 
 ### Mongoose and Apollo
 
-Install mongoose:
+Install Mongoose and dotenv:
 
 ```bash
-npm install mongoose
+npm install mongoose dotenv
 ```
 
 We will imitate what we did in parts [3](/en/part3/saving_data_to_mongo_db) and [4](/en/part4/structure_of_backend_application_introduction_to_testing).
@@ -55,11 +55,12 @@ We also included a few validations. _required: true_, which makes sure that a va
 We can get the application to mostly work with the following changes: 
 
 ```js
-const { ApolloServer, UserInputError, gql } = require('@apollo/server')
+// ...
 const mongoose = require('mongoose')
+mongoose.set('strictQuery', false)
 const Person = require('./models/person')
 
-const MONGODB_URI = 'mongodb+srv://databaseurlhere'
+const MONGODB_URI = process.env.MONGODB_URI
 
 console.log('connecting to', MONGODB_URI)
 
@@ -121,9 +122,10 @@ allPersons: async (root, args) => {
 Apollo server waits for the promise to resolve, and returns the result. So Apollo works roughly like this:
 
 ```js
-Person.find({}).then( result => {
-  // return the result 
-})
+allPersons: async (root, args) => {
+  const result = await Person.find({})
+  return result
+}
 ```
 
 Let's complete the _allPersons_ resolver so it takes the optional parameter _phone_ into account:
@@ -155,7 +157,7 @@ Person.find({ phone: { $exists: false }})
 
 ### Validation
 
-As well as in GraphQL, the input is now validated using the validations defined in the mongoose schema. For handling possible validation errors in the schema, we must add an error-handling _try/catch_ block to the _save_ method. When we end up in the catch, we throw a suitable exception: 
+As well as in GraphQL, the input is now validated using the validations defined in the mongoose schema. For handling possible validation errors in the schema, we must add an error-handling _try/catch_ block to the _save_ method. When we end up in the catch, we throw a exception [GraphQLError](https://www.apollographql.com/docs/apollo-server/data/errors/#custom-errors) with error code : 
 
 ```js
 Mutation: {
@@ -166,8 +168,12 @@ Mutation: {
       try {
         await person.save()
       } catch (error) {
-        throw new UserInputError(error.message, {
-          invalidArgs: args,
+        throw new GraphQLError('Saving person failed', {
+          extensions: {
+            code: 'BAD_USER_INPUT',
+            invalidArgs: args.name,
+            error
+          }
         })
       }
 // highlight-end
@@ -182,8 +188,12 @@ Mutation: {
       try {
         await person.save()
       } catch (error) {
-        throw new UserInputError(error.message, {
-          invalidArgs: args,
+        throw new GraphQLError('Saving number failed', {
+          extensions: {
+            code: 'BAD_USER_INPUT',
+            invalidArgs: args.name,
+            error
+          }
         })
       }
 // highlight-end
@@ -193,6 +203,8 @@ Mutation: {
 }
 ```
 
+We have also added the Mongoose error and the data that caused the error to the <i>extensions</i> object that is used to convey more info about the cause of the error to the caller.
+
 The code of the backend can be found on [Github](https://github.com/fullstack-hy2020/graphql-phonebook-backend/tree/part8-4), branch <i>part8-4</i>.
 
 ### User and log in

src/content/8/fi/osa8c.md

@@ -11,10 +11,10 @@ Laajennetaan sovellusta käyttäjänhallinnalla. Siirrytään kuitenkin ensin k
 
 ### Mongoose ja Apollo
 
-Otetaan käyttöön Mongoose:
+Otetaan käyttöön Mongoose ja asennetaan samalla dotenv:
 
 ```bash
-npm install mongoose
+npm install mongoose dotenv
 ```
 
 Tehdään osien [3](/osa3/tietojen_tallettaminen_mongo_db_tietokantaan) ja [4](/osa4/sovelluksen_rakenne_ja_testauksen_alkeet) tapaa imitoiden.
@@ -54,11 +54,14 @@ Mukana on myös muutama validointi. Arvon olemassaolon takaava _required: true_
 Saamme sovelluksen jo suurilta osin toimimaan seuraavilla muutoksilla:
 
 ```js
-const { ApolloServer, UserInputError, gql } = require('apollo-server')
+// ...
+
+
 const mongoose = require('mongoose')
+mongoose.set('strictQuery', false)
 const Person = require('./models/person')
 
-const MONGODB_URI = 'mongodb+srv://databaseurlhere'
+const MONGODB_URI = process.env.MONGODB_URI
 
 console.log('connecting to', MONGODB_URI)
 
@@ -70,7 +73,7 @@ mongoose.connect(MONGODB_URI)
     console.log('error connection to MongoDB:', error.message)
   })
 
-const typeDefs = gql`
+const typeDefs = `
   ...
 `
 
@@ -109,21 +112,21 @@ Muutokset ovat melko suoraviivaisia. Huomio kiinnittyy pariin seikkaan. Kuten mu
 
 Toinen huomionarvoinen seikka on se, että resolverifunktiot palauttavat nyt <i>promisen</i>, aiemminhan ne palauttivat aina normaaleja oliota. Kun resolveri palauttaa promisen, Apollo server [osaa lähettää vastaukseksi](https://www.apollographql.com/docs/apollo-server/data/resolvers/#return-values) sen arvon mihin promise resolvoituu.
 
-
 Eli esimerkiksi jos seuraava resolverifunktio suoritetaan,
 
 ```js
 allPersons: async (root, args) => {
   return Person.find({})
-},
+}
 ```
 
 odottaa Apollo server promisen valmistumista ja lähettää promisen vastauksen kyselyn tekijälle. Apollo toimii siis suunnilleen seuraavasti:
 
 ```js
-Person.find({}).then( result => {
-  // palautetaan kyselyn tuloksena result
-})
+allPersons: async (root, args) => {
+  const result = await Person.find({})
+  return result
+}
 ```
 
 Täydennetään vielä resolveri _allPersons_ ottamaan huomioon optionaalinen filtterinä toimiva parametri _phone_:
@@ -155,7 +158,7 @@ Person.find({ phone: { $exists: false }})
 
 ### Validoinnit
 
-GraphQL:n lisäksi syötteet validoidaan nyt Mongoose-skeemassa määriteltyjä validointeja käyttäen. Skeemassa olevien validointivirheiden varalta _save_-metodeille täytyy lisätä virheen käsittelevä _try/catch_-lohko. Heitetään catchiin jouduttaessa vastaukseksi sopiva poikkeus, joka on tällä kertaa [UserInputError](https://www.apollographql.com/docs/apollo-server/data/errors/):
+GraphQL:n lisäksi syötteet validoidaan nyt Mongoose-skeemassa määriteltyjä validointeja käyttäen. Skeemassa olevien validointivirheiden varalta _save_-metodeille täytyy lisätä virheen käsittelevä _try/catch_-lohko. Heitetään catchiin jouduttaessa vastaukseksi   [virhekoodilla](https://www.apollographql.com/docs/apollo-server/data/errors/#built-in-error-codes) *BAD\_USER\_INPUT* varustetu poikkeus [GraphQLError](https://www.apollographql.com/docs/apollo-server/data/errors/#custom-errors):
 
 ```js
 Mutation: {
@@ -166,8 +169,12 @@ Mutation: {
       try {
         await person.save()
       } catch (error) {
-        throw new UserInputError(error.message, {
-          invalidArgs: args,
+        throw new GraphQLError('Saving person failed', {
+          extensions: {
+            code: 'BAD_USER_INPUT',
+            invalidArgs: args.name,
+            error
+          }
         })
       }
 // highlight-end
@@ -182,8 +189,12 @@ Mutation: {
       try {
         await person.save()
       } catch (error) {
-        throw new UserInputError(error.message, {
-          invalidArgs: args,
+        throw new GraphQLError('Saving number failed', {
+          extensions: {
+            code: 'BAD_USER_INPUT',
+            invalidArgs: args.name,
+            error
+          }
         })
       }
 // highlight-end
@@ -193,6 +204,8 @@ Mutation: {
 }
 ```
 
+Mongoosen virheen tiedot ja ongelman aiheuttanut data on nyt liitetty poikkeuksen konfiguraatio-olioon <i>extensions</i>, näin ne saadaan välitettyä kutsujalle.
+
 Backendin koodi on kokonaisuudessaan [GitHubissa](https://github.com/fullstack-hy2020/graphql-phonebook-backend/tree/part8-4), branchissa <i>part8-4</i>.
 
 ### Käyttäjä ja kirjautuminen