Giter VIP home page Giter VIP logo

Comments (6)

matrus2 avatar matrus2 commented on May 27, 2024

I know :) The initial purpose of this parameter was for testing.

Let me bring back this one and at the same time add one more e.g. 'esMode' as a replacement in the future.

from dynamodb-stream-elasticsearch.

matrus2 avatar matrus2 commented on May 27, 2024

@GrigoreMihai Nevertheless, I believe you can still use elasticSearchOptions even without testMode flag as AWSConnection adds only aws specific headers to requests to elasticsearch, which are neglected if es is not AWS based. This is why all tests pass with both elasticsearch containers.

from dynamodb-stream-elasticsearch.

GrigoreMihai avatar GrigoreMihai commented on May 27, 2024

@GrigoreMihai Nevertheless, I believe you can still use elasticSearchOptions even without testMode flag as AWSConnection adds only aws specific headers to requests to elasticsearch, which are neglected if es is not AWS based. This is why all tests pass with both elasticsearch containers.

In our case es is AWS based, but we still need to use the auth option as we had setup an user and password for the cluster, I think the tests are not covering this.

from dynamodb-stream-elasticsearch.

matrus2 avatar matrus2 commented on May 27, 2024

You can pass auth option in the same fashion:
https://github.com/matrus2/dynamodb-stream-elasticsearch/blob/master/src/index.js#L45
https://github.com/matrus2/dynamodb-stream-elasticsearch/blob/master/src/index.js#L53
https://github.com/matrus2/dynamodb-stream-elasticsearch/blob/master/src/utils/es-wrapper.js#L14

from dynamodb-stream-elasticsearch.

GrigoreMihai avatar GrigoreMihai commented on May 27, 2024

We tried to use auth as before but with the latest version it is failing because the package is always adding the AWSConnection headers (https://github.com/matrus2/dynamodb-stream-elasticsearch/blob/master/src/utils/es-wrapper.js#L12) and the request will try to use that instead of the specified auth option resulting in some permission errors. ie. specifying the auth option without a flag to remove the default credentials will not work.

from dynamodb-stream-elasticsearch.

tong-bluehill avatar tong-bluehill commented on May 27, 2024

I got same issue. I rollback to v3.0.2. It works now.

from dynamodb-stream-elasticsearch.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.