Mongoose: 새로운 기능: 가상 비동기

현재, 나는 더러운 방법을 사용합니다.

User.virtual('password').set(function(v) {
  this.encryptedPassword = v
})

User.pre('validate', function preValidate(next) {
  return this.encryptPassword().then(next)
})

User.method('encryptPassword', async function encryptPassword() {
  this.passwordSalt = await encryptor.salt()
  this.encryptedPassword = await encryptor.hash(
    this.encryptedPassword,
    this.passwordSalt
  )
})

+1

+1

코드에 아주 좋은 아이디어가 있습니다. 이 문제를 몇 번 보았지만 조사할 시간이 없었습니다. 나는 이 아이디어를 좋아하지만, 다음 출시를 위해 고려할 것입니다.

문제는 .. 사용 구문이 어떻게 생겼습니까?

await (user.password = 'some-secure-password');

작동하지 않습니다.

ECMA262 12.15.4 에 따르면 user.password = 'some-secure-password' 의 반환 값은 _rval_ 이어야 하며 이 경우 'some-secure-password' 입니다.

당신의 반환 값을 가지고 제안하는 someVar = object 가 수 Promise , 그리고에 따라 이 스레드 , 그리고 위의 링크 된 ES262 사양, 그것은이다 "ES 의미 깊은 위반."

또한, 단순히 편의 기능을 갖기 위해 이러한 의미론적 위반 문제를 구현하려는 시도는 상당히 나쁜 생각입니다. 특히 몽구스 코드베이스 전체에 잠재적으로 모든 종류의 나쁜 것을 의미할 수 있기 때문입니다.

그냥 하지 않는 이유:

const hashPassword = require('./lib/hashPassword');

const password = await hashPassword('some-secure-password');
User.password = password; // This is completely normal.

문자 그대로 이렇게 간단한 원 라이너를 위해 처음부터 하면 안 되는 async setter를 만들려고 할 필요가 없습니다.

다음과 같이 할 수도 있습니다.

User.methods.setPassword = async function (password) {
  const hashedPassword = await hashPassword(password);
  this.password = hashedPassword;
  await this.save();
  return this;
};

const myUser = new User();
await myUser.setPassword('mypassword...');

가상 작업, 사전 저장 후크 등을 수행하는 모든 문제를 겪고 있는 이유를 모르겠습니다...

@heisian님 말씀에 동의합니다. 이것은 IMHO에게 기능/API 팽창처럼 느껴집니다. 여기서 인스턴스 메서드를 사용하는 대안이 얼마나 불편한지 모르겠습니다. 그러나 이것에 대해 꽤 중요한 구문 지원을 추가하는 것은 확실히 부풀려진 것처럼 느껴집니다.

다음과 같은 매우 간단한 기능이 있어야 합니다.

User.virtual('password').set((value, done) => {
  encryptValueWithAsyncFunction
    .then(response => done(null, value))
    .catch(reason => done(reason))
  ;
})

@gcanu 당신은 내가 게시한 것을 완전히 무시하고 있으며, 당신이 제안하는 것은 할당 호출에서 Promise를 반환하고 Javascript/ECMA262 사양을 완전히 위반합니다. 코드 스니펫이 작동하려면 setter 함수가 Promise여야 합니다. 이는 정의에 따라 사양에 따라 허용되지 않으며 어쨌든 작동하지 않습니다.

그냥 하는 게 무슨 잘못이야:

await User.setPassword('password');

???

이전에 보지 못한 경우 작동하지 않습니다 .

await (User.password = 'password');

@vkarpov15 이것은 몽구스 관련 문제가 아니라 현재 ECMAScript 사양의 유효성에 대한 질문입니다. 이 "기능 요청"을 닫아야 합니다...

아래 코드는 매우 나쁜 생각입니다! 비밀번호 설정에 save 작업이 포함된 이유는 무엇입니까?

User.methods.setPassword = async function (password) {
  const hashedPassword = await hashPassword(password);
  this.password = hashedPassword;
  await this.save();
  return this;
};

const myUser = new User();
await myUser.setPassword('mypassword...');

몽구스는 더 현대적이고 더 우아한 것이 필요합니다.

@heisian 좋아, 내 실수, 내가 세터 사용을 돌보지 않았습니다 ...

@heisian Plz는 https://github.com/Automattic/mongoose/blob/master/lib/virtualtype.js를 참조

현재 Mongoose IMPL에서 getter 또는 setter 함수를 등록한 다음 호출하면 https://tc39.github.io/ecma262/#sec -assignment-operators-runtime-semantics가 아닙니다. -평가 및 https://github.com/tc39/emascript-asyncawait/issues/82. 그건 달라.

그러니 이 요청을 여십시오.

@fundon , 알려주세요: 가상 세터를 정확히 어떻게 부를까요? 사용법을 보여주세요. async 를 사용하는 경우 약속으로 처리해야 합니다. 원래 예제는 setter/assignment 호출의 어디에도 await 표시하지 않습니다.

내 예제 코드는 단지 예일 뿐입니다... 당신은 또한 이것을 쉽게 할 수 있습니다:

User.methods.setPassword = async function (password) {
  const hashedPassword = await hashPassword(password);
  this.password = hashedPassword;
  return this;
};

const myUser = new User();
await myUser.setPassword('mypassword...');
await myUser.save();

확실히..

당신의 모범은 나에게 좋은 방법이 아닙니다.

원해요

await new User({ password }).save()

더 간단하고 우아한 모드에서 비밀번호를 해시하십시오.

왜요? 몇 줄의 코드를 저장할 수 있습니까? 그 이유는 모든 추가 작업을 정당화하기에 충분하지 않으며 코드베이스에 대한 변경 사항을 깨뜨릴 수 있습니다.

또한 하루가 끝나면 어떻게 표현하든 Mongoose 내에서 내부적으로 진행되는 것은 비동기/대기가 될 수 없는 setter라는 사실을 깨달아야 합니다.

@heisian에 동의하지 않습니다. 몽구스는 오래된 것이 너무 많습니다. 몽구스는 리팩토링이 필요합니다!
몽구스는 현대가 필요합니다.

이 문제가 종료된 경우. Mongoose를 포크하고 리팩토링하겠습니다! 안녕히 계세요!

엄청난! 그것이 오픈소스의 핵심이다. 계속해서 트리밍된 버전으로 포크를 만드십시오. 우리 모두에게 좋을 것입니다.

await (User.password = 'password'); 필요에 대해 걱정할 필요가 없습니다. 유일한 진짜 단점은 user.password = 'password'; 가 발생하는 일부 비동기 작업이 있음을 의미하므로 user.passwordSalt 가 설정되지 않는다는 것입니다. 그것이 후크와 어떻게 관련되는지도 흥미로운 질문입니다. pre('validate') 또는 pre('save') 후크가 있는 경우 user.password 비동기 작업이 완료될 때까지 기다려야 합니까?

나는 이 문제를 손에서 떼고 싶지 않습니다. .save() , .find() 등 뒤에 있는 비동기 동작을 통합하는 데는 많은 가치가 있습니다. 나머지 API와 잘 맞는지 확인하기만 하면 됩니다.

오늘날 비동기 getter와 setter는 나에게 매우 중요합니다. 독점 암호화 방법으로 필드를 복호화/암호화하기 위해 getter 및 setter에서 HTTP 요청을 보내야 하는데 현재로서는 그렇게 할 수 있는 방법이 없습니다. 그것을 달성하는 방법에 대한 아이디어가 있습니까?

@gcanu 나는 그것들을 메소드 로 구현할 것입니다.

언급한 이유와 필요한 비동기 작업을 쉽게 처리할 수 있는 방법이 있다는 사실 때문에 async 동작을 통합하는 데 도움이 되는 유틸리티는 없습니다. 다시 await (User.password = 'password') 는 ECMAScript 규칙을 위반하고 저는 우아하게 구현하는 것이 어렵고 가치가 없다는 것을 보장합니다 ...

당신 말이 맞아, 그 패턴은 우리가 구현할 것이 아니다. 저장하기 전에 비동기 가상이 해결되기를 기다리는 아이디어는 흥미롭습니다.

toJSON({virtuals: true}) 구현에 적합합니다. 직렬화한 후에만 실행하고 싶은 db에 대한 다른 쿼리를 실행하여 얻은 가상 필드 중 일부입니다.

@gabzim JSON.stringify가 약속을 지원하지 않기 때문에 꽤 지저분할 것입니다. 따라서 res.json()은 표현할 추가 도우미를 추가하지 않는 한 비동기 가상을 처리할 수 없습니다.

아 예, 이해가 됩니다. 감사합니다 @vkarpov15

get 콜백 내에서 쿼리를 작성하는 것이 좋은 방법일까요?
나는 이것이 어떤 경우에 유용할 것이라고 생각한다.

Github URL 경로와 같이 문서가 중첩될 수 있는 웹 페이지(또는 문서)의 전체 경로를 얻고 싶다고 가정해 보겠습니다.

const Doc = require('./Doc.js');
//...
subDocSchema.virtual('fullpath').get(async function(){
    const doc = await Doc.findById(this.doc); //doc is a Doc ref of type _id
    return `/${ doc.path }/${ this.path }`
})

쿼리 작업이 비동기식이므로 여기서 async/await를 사용해야 합니다.

@JulianSoto 이 경우 가상이 아닌 메소드 를 사용하는 것을 권장합니다. 가상 을 사용하는 주된 이유 toJSON() 및 toObject() 는 동기식이므로 비동기 가상을 처리하지 않습니다.

나는 이것에 대한 사용 사례도 만났고 좋은 솔루션을 생각하려고 노력하고 있으며 아이디어에 매우 개방적이며 setter 의미를 깨뜨리지 않는다는 데 동의합니다.

JSON 패치 세트를 몽구스 모델에 자동으로 적용하는 유틸리티를 작성했습니다. 깊은 경로로 자동 채우기를 지원합니다: https://github.com/claytongulick/mongoose-json-patch

아이디어는 https://github.com/claytongulick/json-patch-rules 와 같은 몇 가지 규칙과 함께 JSON 패치를 사용하여 '자동' API를 사용하는 것에 매우 근접할 수 있다는 것입니다.

내 계획은 단순 할당이 작동하지 않는 경우 가상을 사용하는 것이었습니다. 패치가 적용되면 가상이 원하는 모든 것을 선택합니다. 이렇게 하면 인터페이스 개체가 실제 몽구스 모델/db 개체와 다를 수 있습니다.

예를 들어, 'payment_methods'에 대한 '추가' 작업을 지원하는 사용자 개체가 있습니다. 지불 방법을 추가하는 것은 배열에 직접 추가하는 것이 아닙니다. 지불 토큰으로 프로세서를 호출하고, 지불 방법 토큰을 다시 받고, 이를 모델에 다른 방식으로 저장하는 등...

하지만 개념적 모델인 인터페이스 모델이 JSON 패치 '추가' 작업으로 패치될 수 있기를 바랍니다.

비동기 세터가 없으면 작동하지 않습니다. 유일한 옵션은 더 나은 아이디어가 없는 한 mongoose-json-patch가 경로, 작업 및 몽구스 메서드 간의 일종의 매핑을 옵션으로 수락하도록 하는 것입니다.

@claytongulick 비동기 작업에 await 대신 비동기 설정 기가 필요한 다음 동기적으로 설정하는 이유는 무엇입니까?

@vkarpov15 기본적 으로 toObject() 및 toJSON() 동기화 하고 toObjectSync() 및 toJSONSync() 기능을 도입하는 것은 어떻습니까? Sync 변형은 async 가상을 건너뛰어야 합니다. (이 패턴은 어딘가 몽구스에서 사용되었던 것으로 기억하는데, 가지고 있어도 이상하지 않을 것이다.)

내 사용 사례는 다음과 같습니다. 다른 모델에서 find() 를 수행하는 가상이 있는 스키마가 있습니다(단순히 ID를 채우는 것보다 조금 더 복잡함). 물론 저장/삭제 후크를 사용하여 기본 모델에 원하는 항목을 비정규화할 수 있지만 유지 관리 비용이 많이 듭니다(이 특정 경우에는 성능 이점이 실제로 필요하지 않습니다). 그래서 가상으로 저를 위해 그렇게 하는 것이 자연스럽습니다.

JSON.stringify() 는 비동기 toJSON() 지원하지 않으므로 불행히도 toJSONSync() 아이디어는 작동하지 않습니다.

find() 가 꽤 복잡하다고 말씀하셨지만 만일을 대비하여 가상 채우기를 살펴보고 싶을 수도 있습니다. 쿼리 미들웨어를 사용해 볼 수도 있습니다.

또한 비동기 가상에는 setter 또는 getter @isamert 만 있습니까?

이 문제가 있는 사람들을 위한 솔루션:

setter만 비동기인 경우 해결책을 찾았습니다. 약간 지저분하지만 잘 작동하는 것 같습니다.
아이디어는 콜백 소품으로 약속 확인자와 설정할 가상 속성을 포함하는 객체를 가상 setter에 전달하는 것입니다. setter가 완료되면 콜백을 호출합니다. 이는 외부에 개체를 저장할 수 있음을 의미합니다.

첫 번째 질문에서 영감을 얻은 기본 예를 사용하려면:

const User = new Schema(
  {
    username: {
      type: String,
      index: true,
      unique: true
    },
    encryptedPassword: {
      type: String,
      required: true,
      minlength: 64,
      maxlength: 64
    }
})

User.virtual('password').set(function generate(inputWithCb, virtual, doc) {
  let cb = inputWithCb.cb;
  let password = inputWithCb.password;
  encryptor.hash(password)
  .then((hash) => {
    doc.set("encryptedPassword", hash);
    cb && cb();
  });
})

// create the document
const admin = new User({
  username: 'admin'
});
// setup the promise for setting the async virtuals
const pwdProm = new Promise((resolve) => {
  admin.set("password", {cb: resolve, password: "admin"});
})

//run the promise and save only when the virtual setters have finished executing
pwdProm
.then(() => {
  admin.save();
});

이것은 원치 않는 결과를 초래할 수 있으므로 자신의 책임하에 사용하십시오.

@silto 그냥 약속을 반환하는 스키마 메서드를 사용하지 않는 이유는 무엇입니까?

@vkarpov15 보통 그렇게 하겠지만, 내가 한 프로젝트에서는 json "계획"에서 자동으로 생성된 스키마, 가상 및 graphQL 끝점이 있으므로 특정 경우에 대한 방법 대신 균일한 가상 인터페이스를 사용하는 것을 선호합니다.

@silto 코드 샘플을 제공할 수 있습니까? 나는 이것이 어떻게 생겼는지 보고 싶다

setter의 경우 저장을 하거나 그냥 문서 데이터를 찾아보고 저장하면 프라미스이므로 프라미스인 필드를 확인하고 해결한 후 저장할 수 있습니다.

데이터를 찾고 싶다면 스키마 옵션으로 이 Model이 Promise 유형이 되도록 정의하거나 모델을 생성할 때 스키마를 확인하고 Promise인 setter, getter 또는 virtual이 있는지 확인한 다음 그것을 약속으로.

또는 이미 가지고 있는(execPopulate) 함수와 같은 exec를 간단히 사용할 수 있습니다.

이력서에서 setter, getter 또는 virtual이 있는 데이터를 관찰하려는 경우 해당 문제에 대한 함수를 빌드할 수 있습니다. 데이터를 저장하려면 이미 약속이므로 동일한 함수를 사용하여 데이터를 변환할 수 있습니다. 저장하기 전에.

나는 약속과 함께 가상을 사용하지만 명시적 약속을 사용할 때 거의 항상 약속에 신경 쓰지 않지만 어떤 경우에는 Doc을 사용합니다..then(), 나는 약속이 있는 setter를 사용한 적이 없기 때문에 그 문제가 없습니다...

어느 쪽이든 모든 데이터를 이미 확인하고 약속된 setter를 정의한 후 모든 약속된 가상 및 게터에서 "then"을 사용할 필요가 없는 일종의 래퍼가 있으면 좋을 것입니다.

이 접근 방식을 도와드릴 수 있기를 바랍니다.

친애하는.

추신: Promisifed virtuals 사용의 일반적인 예, 제 경우에는 2개의 경로를 사용하여 내 문서를 삭제하거나 외부 데이터에 따라 업데이트할 수 있는지 확인합니다. . 앞서 말했듯이 명시적 약속이 이 문제를 해결해 주지만 이러한 작업을 수행할 수 있는지 내부적으로 확인하려면 먼저 이 약속을 해결해야 했습니다.

@chumager 몇 가지 코드 샘플을 제공할 수 있습니까?

안녕하세요, 예를 들어, 아래 내 의견에 따르면 2개의 가상 _update 및 _delete와 스키마에 정의되어 있지 않은 경우 해당 가상을 정의하는 플러그인을 사용하여 true를 반환합니다.

크레딧을 정의하는 시뮬레이션 모델과 mkt 데이터로 시뮬레이션을 게시하는 프로젝트 모드가 있습니다.
시뮬레이션과 연결된 프로젝트가 있는 경우 시뮬레이션을 삭제할 수 없으며 프로젝트가 투자용으로 게시된 경우 업데이트할 수 없습니다.

시뮬레이션에서 가상 _update의 해결 방법은 시뮬레이션이 참조되고 상태가 "En Financiamiento"인 프로젝트를 찾는 것입니다. 이 쿼리가 참이면 시뮬레이션을 업데이트할 수 없습니다... 분명히 "찾기"는 약속, 그래서 가상도 ...

일반적으로 프론트엔드에서 이 가상을 사용하는 것처럼 데이터는 객체를 해결하는 모듈에 의해 구문 분석됩니다(co 또는 express-promise에 따라 하나 또는 결과 배열).

문서를 보고 싶은 경우 내 가상이 약속이라는 것을 알게 될 것이므로 co 모듈을 사용하여 해결해야 하지만 이미 결과를 약속으로 사용해야 했습니다... 아마도 결과에 co를 추가하기만 하면 됩니다. 마법을 수행하거나 찾기 후에 co를 사용하는 플러그인으로 ... 하지만 결과 세트가 이미 작업을 완료한 것처럼 더 자연스럽게 보입니다.

나는 몽구스에서 데이터를 가져오기 위해 많은 끝점을 사용합니다. si 나는 그 기능을 어디에서나 사용하거나 찾기를 위해 포스트 후크를 사용해야 할 것입니다.

getter와 마찬가지로 후크는 사전 유효성 검사를 수행해야 하지만 문서에 순환 참조 및 생성자와 같은 기타 소품이 있기 때문에 문서의 다른 소품을 건드리지 않는 것이 중요합니다.

문안 인사...

추신: 예제 codo가 정말로 필요하면 저에게 알려주십시오.

@chumager 산문의 큰 벽 !== 코드 샘플. 코드 샘플을 정말 선호합니다.