This repository has been archived by the owner on May 7, 2021. It is now read-only.
Update logger #2054
Merged
Update logger #2054
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
justinr86
requested review from
ngosset,
khalidelaggan,
sdzhangtao,
DianeLiu2019,
davidheerema and
kevinyang06
davidheerema
previously requested changes
Jun 25, 2020
There was a problem hiding this comment.
@justinr86 You will see in the PR template there is a check to maintain our list of environment variables:
" I have updated the azurescript.sh with any new environment variables and added them to the appsettings"
Please update that as well - two locations in that file when we add new environment variables 1) define the variable and 1) apply the variable down in the script.
sdzhangtao
approved these changes
Jul 7, 2020
justinr86
dismissed
davidheerema’s stale review
Change request has been addressed.
ngosset
pushed a commit
that referenced
this pull request
Jul 9, 2020
* Validate date (#1799) * reformat postal code * add tests for postal code validation * add more validation on date in howdiditstart page, * make comment more clear * change "==" to "===" * add test to make sure the validate date part is work * corrct fr.json error Co-authored-by: Steve Astels <sastels@gmail.com> * adding gtm tag to build pipeline (#1815) * Sanitize html tags out of user inputs (#1773) * phase banner tag more obvious (#1816) * add prodVersion to report (#1768) * Updated business form (#1802) * added field for name of business * added all fields for business form * update confirmation summary page with new business form * added french translation * added new business fields for analyst email * re-implemented radio button list * added help text and fixed translation Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> Co-authored-by: amazingphilippe <amazingphilippe@gmail.com> * Refactor form default values (#1810) * Error messages (#1822) * Error messages * changed saisir to entrez * Anonymous flow opt-in (#1820) * change report anonymous from picklist to checkbox and added all necessary translation * Update f2/src/locales/en.json Co-Authored-By: Steve Astels <sastels@users.noreply.github.com> * Update f2/src/locales/en.json Co-Authored-By: Steve Astels <sastels@users.noreply.github.com> * Update f2/src/locales/fr.json Co-Authored-By: Steve Astels <sastels@users.noreply.github.com> * removed unsed translation * remove unused translation * remove unused translation * remove unused translation * modified test.js to allow submit without select a option * setting new anonymous variable in location page modified analyst email * removed unused imports * modified test.js to adapted new anonymous field * anonyous opt-in flow. Fixed email errors, data handling and naming conventions * fixed analystemail test * fixed translations * translation fix Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> Co-authored-by: Steve Astels <sastels@users.noreply.github.com> Co-authored-by: Philippe Caron <amazingphilippe@gmail.com> * Remove Debugging Logging (#1832) * Updating env.example with new tags (#1814) * Adding GTM ID * Adding GTM var * Server side field validation (#1823) * fix: upgrade @chakra-ui/core from 0.5.2 to 0.7.0 (#1837) Snyk has created this PR to upgrade @chakra-ui/core from 0.5.2 to 0.7.0. See this package in NPM: https://www.npmjs.com/package/@chakra-ui/core See this project in Snyk: https://app.snyk.io/org/cds-snc/project/232bd9dd-228e-4143-add6-b03510d9d062?utm_source=github&utm_medium=upgrade-pr Co-authored-by: Steve Astels <sastels@users.noreply.github.com> * fixed translation on business summary (#1829) * Anonymous summary (#1834) * change report anonymous from picklist to checkbox and added all necessary translation * Update f2/src/locales/en.json Co-Authored-By: Steve Astels <sastels@users.noreply.github.com> * Update f2/src/locales/en.json Co-Authored-By: Steve Astels <sastels@users.noreply.github.com> * Update f2/src/locales/fr.json Co-Authored-By: Steve Astels <sastels@users.noreply.github.com> * removed unsed translation * remove unused translation * remove unused translation * remove unused translation * modified test.js to allow submit without select a option * setting new anonymous variable in location page modified analyst email * removed unused imports * modified test.js to adapted new anonymous field * anonyous opt-in flow. Fixed email errors, data handling and naming conventions * fixed analystemail test * fixed translations * Anonymous summary * linted and added test for anon summary Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> Co-authored-by: kevinyang06 <58781425+kevinyang06@users.noreply.github.com> Co-authored-by: Steve Astels <sastels@users.noreply.github.com> * change report date format and self harm words message * delete prodVersion after using in report * delete form data populates empty report with initialstate (#1839) * less textboxes (#1825) * guidance on what happened page * removed tell us more fields in forms, summaries, analyst email, translations * removed tellusmore fields in defaultValues * imported formDefaults * format suspect and victim phone numbers * fixing translations (#1848) * uncamel can now space out camel strings with numbers, fixed businessSize camelcased value (#1847) * format submitTime as well * Duplicate information is displayed in the analyst email (#1850) * deleted two fields * one deletion will be done in defaultValue.js Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * fix French url (#1852) * number of submissions and availability not rolling over with new day (#1838) (#1846) * add validation for date of Feb in leap year and non-leap year (#1809) * tweaks to thank you page (#1853) * deploy storybook to github pages (#1781) * deploy storybook to github pages Co-authored-by: Philippe Caron <amazingphilippe@gmail.com> * update changelog to v1.1.0 prod release (#1843) * Move submitted and reportId out of formData (#1863) * Fix storybook CI deploy (#1870) * Changing log from error to info (#1878) * Load testing scripts (#1856) * This is the commit message * added missing fields by iterate through defaultForms in confirmation page (#1895) Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> Co-authored-by: ngosset <norman.gosset@cds-snc.ca> * check anonymous default correctly (#1883) * Reports Submitted counter not initialized after reboot (#1865) * modified getReportCount() to set number of submission * fixed number of submission = o in isAvailable * removed unused code Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * added StateProvider to themeDecorator (#1866) * Date validation util (#1889) * add validateDate.js and validateDate.test.js components to reuse. add date-fns package for date validation test. * use date-fns * working validateDate util * tests, updated forms, updated error and form array control to accept array-based error messages * updated Error summary documentation Co-authored-by: Diane Liu <deyuan.liu@cds-snc.ca> * Stop accepting zero bytes attachments. (#1891) * Stop accepting zero bytes attachments. * changed == to === Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> Co-authored-by: dhcdn <51758091+davidheerema@users.noreply.github.com> * New e2e Tests - Merging Hong's code (#1841) * Merging Hong's code in a new branch * April22 * changedate * Another A11y hock Co-authored-by: lmcbhvu <60269104+lmcbhvu@users.noreply.github.com> * Added more Cypress CLI tests - A11y checker (#1907) * Merging Hong's code in a new branch * April22 * changedate * Another A11y hock * More CLI tests added * Changed targeted Env Co-authored-by: lmcbhvu <60269104+lmcbhvu@users.noreply.github.com> * give victim more clear information about the file type which can be uploaded (#1904) * give victim more clear information about the file type which can be uploaded. * change fontsize for the list item * bring the list item closer * group the list in a box to bring spacing even better * Logged more detailed error message if content moderator has a problem (#1890) * Logged more detailed error message if content moderator has a problem * Fixed one code error. The variable is error, not err * Log the stack trace as well. error this time is an exception * Only send the files to content moderator for supported types. * Change log lines a smidge Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> Co-authored-by: dhcdn <51758091+davidheerema@users.noreply.github.com> * Improve headers (#1898) * configure content security policy * add referrer policy * add feature policy * change field name of Cryptocurrency (#1912) Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * add warning message in email subject if self harm words detected (#1914) * Enforce consent better (#1908) * commit for debug * commit for debug * commit for debug * enforce consent is working for howDidItStart page * apply enforce to consent page to all forms * test passed for location page * apply test file change to all forms * fixed compile error * fixed test files run forever issue * removed unused code * anonymous page for enfore consent Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * Use winston and structured logging (#1909) * Use winston and structured logging * package lock json file * removed unused imports * Fixed the issue of Error object can't be transformed to JSON, results in empty {}' * Not to stringify content moderator Error object Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> Co-authored-by: dhcdn <51758091+davidheerema@users.noreply.github.com> * set report language with english or french instead of en/fr (#1915) Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> Co-authored-by: dhcdn <51758091+davidheerema@users.noreply.github.com> * Added a TOTP code to bypass submission per day restriction (#1911) * Added a TOTP code to bypass submission per day restriction * Missing env var from example env file * Added util to generate TOTP secret * Comments and stuff * Fixed bug where we are setting numberOfSubmissions to undefined Co-authored-by: dhcdn <51758091+davidheerema@users.noreply.github.com> * add express-winston for http request logging (#1916) * add express-winston for http request logging * Log remoteIP and requestSize Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> Co-authored-by: dhcdn <51758091+davidheerema@users.noreply.github.com> * enable victim to remove the post code value in location page … (#1917) * enable victim to remove the post code field in locaiton page when they go back from confirmation page by clicking the edit button * delete unused line * delete unwanted space * remove the phone number when the call center is closed (#1928) * remove the phone number when the call center is closed due to the pandemic * remove the "report by phone" whole section * Renew privacy statement (#1930) * add "web analytics" section for the privacy statement * remove "authorization to collect your information" part * add "authorization to collect your information" section back * Updated changelog for version 1.3 (#1935) Squash and merge without approvals - It's just a text file * Enforce content page after cancel (#1927) Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> * add background color yellow for the self harm word in the email body (#1939) * add background color yellow for the self harm word in the email body * fix test issue * Add Storybook documentation re the new form defaults file (#1919) * added description for default value.js * added example of formDefaults Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * change how data is handled when page is loaded (#1946) Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * cancel a report and then use back button go go back to the report flow). This PR make the app redirect the user back to the consent page (#1944) * Disable FTP(S) (#1941) Policy dictates that we should only allow secure FTP communication. We will take it a step further and disable it entirely. This has already been done to our current active environments. * add link for "RCMP Personal Information Databank Operational Case Records (RCMP PPU 005)", also change the French version for last sentence to match the english version (#1945) * useEffect to remove the save state warning (#1921) * useEffect to remove the save state warning * removed logging Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> * change links for stay up-to-date on what scams are most common. (#1955) Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * redirect user to privacy consent page insted of report flow if user (#1958) cancel report in anonymous page * Phantom button (#1948) * Update CSS to limit width of input elements * Update following comments. Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> * set doneForms to false when report cancelled (#1960) Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> * Use jimp to transform png to jpg (#1951) * Use jimp to transform png to jpg * fixed linit warning * convert png to jpg before save to blob * revert back * copy over scanning results to converted images Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> * Nbsp added to french text. (#1967) * add call link to all the phone numbers in the app (#1963) * add call link to all the phone number in ThankYouPage and PrivacyStatementPage * add call link in landing page for 911 * fix the space and period * delete unwanted blank line * Stop server from accepting zero-sized files (#1964) * Stop server from accepting zero-sized files * add zero size test Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> * App version being overriden (#1938) Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> * add dstore file to gitignore, remove exsisting files that were checked in (#1936) Merging * Merge Hong's changes into new branch (#1940) * Merge Hong changes into new branch * Github merge training for RCMP QC * John recaptcha (#1899) * play with reCaptcha * Added recpatcha verification * google recpatcha server side verification * Fetch API to call google reCaptcha verify * check google recaptcha at server side, save score in state * removed await, take full advantage of async when calling google recaptcha, also enhanced error handling * Use isomorphic-unfetch instead of isomorphic-fetch * merge conflict Co-authored-by: Steve Astels <sastels@gmail.com> Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> Co-authored-by: Steve Astels <sastels@users.noreply.github.com> * Revert "John recaptcha (#1899)" (#1973) This reverts commit 6d01fe4. * Provide the city/municipality and prov/terr on the Analyst Report when given the postal code (#1969) * code doesn't work * added city and prov based on postal code in analyst emial * fixed package.json error * keep fr.json align with master * keep fr.json align with master Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> Co-authored-by: ngosset <norman.gosset@cds-snc.ca> * Remove is checked (#1966) * isChecked removed * Removed remaining isChecked * Removed unused recurrenceCheck variable. Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> * Redirect requests coming from invalid referer. (#1974) Co-authored-by: ngosset <norman.gosset@cds-snc.ca> * Error in reports when postal code is omitted (#1987) * fixed analystEmail bug: * update fr.json with master * update fr.json with master branch Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * add warning banner more distracting (#1978) * chang French link word to match english version. (#1961) * chang French link word to match english version. * change the the word link according documentation Co-authored-by: ngosset <norman.gosset@cds-snc.ca> * Updated the hard coded version for June-4 release (#1991) Co-authored-by: khalid <khalo> Co-authored-by: ngosset <norman.gosset@cds-snc.ca> * Removing the dev App version notation (#1996) * Updated the hard coded version for June-4 release * Removing the dev notation Co-authored-by: khalid <khalo> * Encrypted E-mail and attachments redesign (#1975) * Generate a SAS for each file upload and print to console * sas * Attachments replaced with download link * Encrypt files before saving to SAS * E-mail the report as an attachment because groupwise has awful HTML e-mail support * Delete 000146484.cer.pem * Remove accidental checkins * Remove accidental checkins * Remove accidental checkins * Remove accidental checkins * Undo stripping of nbsp from fr.json * . * Fixing how we send smime e-mail * Moved subject line to openssl smime command. Pass nodemailer the RAW output of openssl * Comment * Fixed fr.json label that disappeared. Removed unused import * Don't send the png in the report. Don't encrypt if run without HRMIS ids in env var * JS-ify my code - misc cleanup * Add reminder for env vars (#2002) * Updating Environment Variables (#2001) * Stop Colorizing WinstonExpress (#1994) * Remove word Error from debug Log (#1977) * React warning (#1990) * Initial work, tabindex added to components. * Removed ':javascript' from components. * Update changelog (#2006) * Cancelpagelink (#1995) * chang version to 1.4.0 * change link on help resource as CAFC requested * New john recaptcha (#1981) * added back landingPage labels * added back labels * reapply google recaptcha Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> * make link open in new tabs (#2005) Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * Cosmetic error (#2009) * specify the error message when upload file type is not supported * file extension can be capital case. * change back to .doc * undo the change not belong to this PR * undo the change not belong to thiS PR * fix cosmetic issue on business page and review page * undo the change that not belong to this PR * Remove CSP for now (#2012) * Filetype (#2008) * specify the error message when upload file type is not supported * file extension can be capital case. * change back to .doc * undo the change not belong to this PR * undo the change not belong to thiS PR * converting the file extension to lower case if user upload file with capital case extension * deleted commented line * Keyword fine tuning (#2018) * Initial work, break up text into array and analyze * Key word search now only matches complete words * upgrade Dockerfile node from 12.15 to 12.18 (#2017) Co-authored-by: ngosset <norman.gosset@cds-snc.ca> * If a user enters an invalid postal code, the report to the analyst is malformed. (#2034) * surrounding postalcode look up with try/catch * fine tune the code Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * USE getFileExtension function from filenameUtils.js to consist with (#2024) other.js file for easiler maitainance. * fix: upgrade @azure/storage-blob from 12.1.1 to 12.1.2 (#2028) Snyk has created this PR to upgrade @azure/storage-blob from 12.1.1 to 12.1.2. See this package in NPM: https://www.npmjs.com/package/@azure/storage-blob See this project in Snyk: https://app.snyk.io/org/report-a-cybercrime/project/e4490367-7941-440f-bd6b-97f5a82818c3?utm_source=github&utm_medium=upgrade-pr Co-authored-by: ngosset <norman.gosset@cds-snc.ca> * Analyst group email (#2025) * added back landingPage labels * added back labels * encrypt email for multiple recipients and send to a group email * move analyst group email into env file Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> Co-authored-by: ngosset <norman.gosset@cds-snc.ca> * fix: upgrade prettier from 2.0.2 to 2.0.5 (#2043) Snyk has created this PR to upgrade prettier from 2.0.2 to 2.0.5. See this package in NPM: https://www.npmjs.com/package/prettier See this project in Snyk: https://app.snyk.io/org/report-a-cybercrime/project/e4490367-7941-440f-bd6b-97f5a82818c3?utm_source=github&utm_medium=upgrade-pr Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> * Change the "return to previous version" link to the "Choose a secure Sign-In Method" Page (#2015) * change link in the banner * open link in the same page Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * fix: upgrade axe-core from 3.5.2 to 3.5.3 (#2026) Snyk has created this PR to upgrade axe-core from 3.5.2 to 3.5.3. See this package in NPM: https://www.npmjs.com/package/axe-core See this project in Snyk: https://app.snyk.io/org/report-a-cybercrime/project/e4490367-7941-440f-bd6b-97f5a82818c3?utm_source=github&utm_medium=upgrade-pr Co-authored-by: ngosset <norman.gosset@cds-snc.ca> Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> * fix: upgrade cypress from 4.3.0 to 4.6.0 (#2030) Snyk has created this PR to upgrade cypress from 4.3.0 to 4.6.0. See this package in NPM: https://www.npmjs.com/package/cypress See this project in Snyk: https://app.snyk.io/org/report-a-cybercrime/project/e4490367-7941-440f-bd6b-97f5a82818c3?utm_source=github&utm_medium=upgrade-pr Co-authored-by: ngosset <norman.gosset@cds-snc.ca> Co-authored-by: khalid elaggan <khalidelaggan@users.noreply.github.com> * fix: upgrade axe-core from 3.5.3 to 3.5.4 (#2046) Snyk has created this PR to upgrade axe-core from 3.5.3 to 3.5.4. See this package in NPM: https://www.npmjs.com/package/axe-core See this project in Snyk: https://app.snyk.io/org/report-a-cybercrime/project/e4490367-7941-440f-bd6b-97f5a82818c3?utm_source=github&utm_medium=upgrade-pr Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> * Add groupemail env to script (#2044) * add more help resource link (#2040) * change link to help resource * add more help resource link on thank you page * Key word refinement (#2041) * Temp commit * Added natural language processing, refactored code * Added comments * Add package-lock Co-authored-by: ngosset <norman.gosset@cds-snc.ca> * Improve logging (#2036) * Formatted output, consolidated code, added log to file * Fixed unnecessary import. * Added environment variables and logic. * Update package-lock * Bookmark reload (#2020) * added back landingPage labels * added back labels * allow direct loading of arbitrary pages Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> Co-authored-by: ngosset <norman.gosset@cds-snc.ca> Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> * remove The City and Province base on the postal code when report anonymous (#2045) * remove The City and Province base on the postal code in the analyst email if the report is an anonymous * remove comment line * fixed postal code look up Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> Co-authored-by: kevinyang06 <58781425+kevinyang06@users.noreply.github.com> * open a new brower after clicking the link (#2060) * Attach stemmer based on language (#2065) * change version to 1.5.0 (#2057) Co-authored-by: ngosset <norman.gosset@cds-snc.ca> * required contact fields (#1902) * required contact fields * fixed missing key validation * fixed validation logic * remove log Co-authored-by: kevinyang06 <58781425+kevinyang06@users.noreply.github.com> * Resolve babel deprecation warning (#2064) * Resolve production error, update logs (#2076) * Resolve production error, update logs * Resolve test failure * Elaborate Availability Logging & Remove duplicate logic (#1999) * Elaborate Availability * Remove duplicate check for last request * Moved direct access check to checkIfAvailable * Addedlog messages and comments * Moved availableData variables to checkIfAvailable * Moved getReportCount into async function. * Clean up code. * Resolve errors after merge * Resolve error when using TOTP secret Co-authored-by: Justin <richardsonj1986@gmail.com> Co-authored-by: justinr86 <62246403+justinr86@users.noreply.github.com> Co-authored-by: ngosset <norman.gosset@cds-snc.ca> * If the user submitted the report in French, the analyst report should be in French (#2042) * set email language is working * not completed yet * repleace names from locale * added trans for file upload * added more translation for analyst report email * added missing trans in en/fr.json * added translations * added translation for all section heading * added translation for self harm word * added translation for file upload error message * modfied testing case * modified jest test case * fixed lint error * modified scanFiles to get language * added translation for noFiles Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> * Update logger (#2054) * Add option for prettyPrint, raw JSON is default * Add option for prettyPrint, raw JSON is default * Updated Azure script * specify the error message when upload file type is not supported (#2007) * specify the error message when upload file type is not supported * tell user what file types are supported in the error message * fix: upgrade storybook-addon-jsx from 7.1.15 to 7.2.3 (#2027) Snyk has created this PR to upgrade storybook-addon-jsx from 7.1.15 to 7.2.3. See this package in NPM: https://www.npmjs.com/package/storybook-addon-jsx See this project in Snyk: https://app.snyk.io/org/report-a-cybercrime/project/e4490367-7941-440f-bd6b-97f5a82818c3?utm_source=github&utm_medium=upgrade-pr Co-authored-by: ngosset <norman.gosset@cds-snc.ca> Co-authored-by: kevinyang06 <58781425+kevinyang06@users.noreply.github.com> * Resolve duplicate code * Update with changes from master * Update with latest from master Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com> Co-authored-by: Steve Astels <sastels@gmail.com> Co-authored-by: dhcdn <51758091+davidheerema@users.noreply.github.com> Co-authored-by: Steve Astels <sastels@users.noreply.github.com> Co-authored-by: amazingphilippe <amazingphilippe@gmail.com> Co-authored-by: kevinyang06 <58781425+kevinyang06@users.noreply.github.com> Co-authored-by: Kevin Yang <kevinyang@Kevins-RCMP-MacBook-Pro.local> Co-authored-by: Snyk bot <snyk-bot@snyk.io> Co-authored-by: davidheerema <david.heerema@cds-snc.ca> Co-authored-by: ngosset <norman.gosset@cds-snc.ca> Co-authored-by: Diane Liu <deyuan.liu@cds-snc.ca> Co-authored-by: John Zhang <john.zhang.cds@gmail.com> Co-authored-by: John Zhang <zhangtao.ottawa@gmail.com> Co-authored-by: khalid elaggan <khalidelaggan@users.noreply.github.com> Co-authored-by: lmcbhvu <60269104+lmcbhvu@users.noreply.github.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #2053
Description
Winston Logger now defaults to raw JSON. Use environment variable LOGGING_PRETTY_PRINT to have the formatted JSON written to console.
Any new packages installed?
N/A
Required followup work
Add environment variable LOGGING_PRETTY_PRINT to enable formatted log messages. This is only required for devs to make reading log messages easier.
Checklist: