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.

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

_ensureIndex has been deprecated #832

Closed
polygonwood opened this issue Jan 24, 2022 · 0 comments · Fixed by #838
Closed

_ensureIndex has been deprecated #832

polygonwood opened this issue Jan 24, 2022 · 0 comments · Fixed by #838

Comments

@polygonwood
Copy link

I'm having an issue:

  • Give an expressive description of what is went wrong
    startup of application shows these messages :
    ...
    [FilesCollection.storagePath] Set to: assets/app/uploads/uploadedFiles
    D20220124-20:50:03.195(1) (collection.js:743) _ensureIndex has been deprecated, please use the new 'createIndex' instead, index: {"createdAt":1}
    ...

  • Version of Meteor-Files you're experiencing this issue ostrio:files 2.0.1

  • Version of Meteor you're experiencing this issue 2.6-beta.4

  • Where this issue appears? OS (Mac/Win/Linux)? Browser name and its version? Linux

  • Is it Client or Server issue? - Server

  • Post Client and/or Server logs with enabled debug option, you can enable "debug" mode in Constructor

I had an issue and I've solved it on my own:

  • Provide a description of steps that you've followed to solve the problem
  • Provide links to websites and/or pages with the information that helped you (if there are any)

I have a suggestion:

  • Describe your feature / request : adapt mongo 5 createIndex as suggested
  • How you going to use it? Give a usage example(s)

Documentation is missing something or incorrect (have typos, etc.):

  • Give an expressive description what you have changed/added and why
  • Make sure you're using correct markdown markup
  • Make sure all code blocks starts with tripple ``` (backtick) and have a syntax tag, for more read this docs
  • Post your addition/changes in issue, we will manage it

Thank you, and do not forget to get rid of this default message

dr-dimitru added a commit that referenced this issue Jun 9, 2022
__Major changes:__

- ⚠️ `FSName` now sanitized and limited to 28 symbols, no white-spaces allowed
- ⚠️ `fileId` now sanitized and limited to 20 symbols, no white-spaces allowed
- ⚠️ File extension now sanitized and limited to 20 symbols, no white-spaces allowed
- 📦 Decouple `fs-extra`, use native node.js `fs` module
- 👨‍💻 Add missing `isData` helper via #795, by @harryadel

__Changes:__

- 👷‍♂️ Fix: #827, thanks to @aertms
- 👷‍♂️ Fix: #832, thanks to @polygonwood
- 👷‍♂️ Fix: #813, thanks to @Prinzhorn
- 👨‍💻 Fix: #834, thanks to @bladerunner2020
- 📔 Fix: #803, thanks to @bartenra
- 🏗 Sanitize inputs (`FSName`, `fileId`, `extension`) to avoid command injection
- 📔 Improved documentation, special thanks to @Prinzhorn and @make-github-pseudonymous-again
- 📔 GridFS documentation fix: #818
- 👷‍♂️ Utilize `Meteor._debug` where possible instead of `console.log`

__Other:__

- 🤝 Compatibility with `meteor@2.7.3`
@dr-dimitru dr-dimitru mentioned this issue Jun 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants