Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update mongodb dependency #6

Open
wants to merge 56 commits into
base: master
Choose a base branch
from
Open

Update mongodb dependency #6

wants to merge 56 commits into from

Conversation

zhao-li-us
Copy link

Hi @jankleinert ,

Thank you for writing up this tutorial:
https://developers.redhat.com/articles/deploy-a-NodeJS-app-OpenShift

and providing this example application.

I was wondering if you are willing to accept a pull request to update the Mongo DB library in this example application.

When I was working through the tutorial, I ran into a FIPS error which is resolved by upgrading the MongoDB library. Unfortunately, the newer MongoDB library used a slightly different syntax.

I ended up needing to upgrade the syntax for all of the routes.

At the end of the exercise, the MongoDB library was updated and the example application code was refactored to work with the latest MongoDB library.

If you are interested in updating the example code to use the newer library and syntax, then I can clean up this PR or submit a separate PR with less noise and solely focused on upgrading MongoDB library.

Thanks again for writing all of this up. 🙏

@zhao-li-us
update meta information to test effect
48b8d0c
@zhao-li-us
update mongodb dependency version
51ee135
@zhao-li-us
add in output logs
e210944
@zhao-li-us
add more debugging outputs
aa50aad
@zhao-li-us
output actual connection string used
33e59cb
@zhao-li-us
fix connection string output
9523b3d
@zhao-li-us
add a test route that does not require database connection
01c8563
@zhao-li-us
add more debug outputs
27cf458
@zhao-li-us
add more debug output
e53bac1
@zhao-li-us
add more output statements to test if connection ended
a1991c4
@zhao-li-us
log out connection details
8ca001e
@zhao-li-us
update mongo db debug code to latest API
f14d7ca
@zhao-li-us
fix typo
fe0917a
@zhao-li-us
fix debug statement error
a69a716
@zhao-li-us
investigate collections
360aa7d
@zhao-li-us
try .toArray method to list out collection
1407127
@zhao-li-us
test if older API still works
b25882f
@zhao-li-us
try to hold off on closing connection
ed4c4cf
@zhao-li-us
fix waiting code
42863d0
@zhao-li-us
try using await to assign variable
7ab16ac
@zhao-li-us
clean up debugging routes
22239da
@zhao-li-us
fix collections variable scope issue
aaf2cd2
@zhao-li-us
ensure database call is finished
4f710be
@zhao-li-us
fix variable scope
f2997f7
@zhao-li-us
add debug statements to troubleshoot collections
eaa262f
@zhao-li-us
try using a more global variable to ensure data gets saved properly
a98338c
@zhao-li-us
add troubleshooting statements
2d9e161
@zhao-li-us
move response command into callback to try to avoid scoping issues
a5e2c03
@zhao-li-us
try returning only a list of collection names only
e12e845
@zhao-li-us
update all orders action to avoid variable scope issues
07d9946
@zhao-li-us
clean up debug routes
4b07405
@zhao-li-us
remove debug statements that are no longer needed
1b61972
@zhao-li-us
refactor application route
6c26da9
@zhao-li-us
remove extraneous debug statements
d328feb
@zhao-li-us
remove old syntax
62a6b44
@zhao-li-us
try to fix cursor.count error
a9452aa
@zhao-li-us
troubleshoot ticket route by adding statements in one at a time
fde3aab
@zhao-li-us
add next statement in
3643332
@zhao-li-us
try to find which chained method is causing deprecation error
586e451
@zhao-li-us
test outer loop
ccecbea
@zhao-li-us
fix issue with using .then syntax
4fdff7f
@zhao-li-us
move forward with await syntax
64fd19f
@zhao-li-us
fix variable name typo
1d7ac17
@zhao-li-us
try await
330007b
@zhao-li-us
remove reference code that is no longer needed
49dfefd
@zhao-li-us
move away from callback syntax
0dcc4a2
@zhao-li-us
add in ordering list of orders
f667f85
@zhao-li-us
try to get highest ticket number and finish request
94c0ac9
@zhao-li-us
wait for insertion to complete
1a2c5d0
@zhao-li-us
clean up comments and check if old code is actually useful
1a75e3a
@zhao-li-us
add code for first ticket using new syntax
624f8d4
@zhao-li-us
update callback code to use await syntax to catch error
eb4e1f6
@zhao-li-us
try catching error from duplicate ID
0dbfac6
@zhao-li-us
remove mongo shell syntax
cc5bb90
@zhao-li-us
clean up debug statements and whitespace
b39451f
@zhao-li-us
clean up: fix white space, reorder routes, reorder variable definitio…
ff9032f 
…ns, add comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@zhao-li-us