Skip to content
This repository was archived by the owner on Dec 5, 2025. It is now read-only.

Conversation

@SouadHadjiat
Copy link
Member

@SouadHadjiat SouadHadjiat commented Jan 24, 2025

Proposed changes

  • Hide token by default and return request headers copy

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality

Further comments

to test locally, you need first to set json_logging: true in your config.yml, then you'll need to simulate a ping error on opencti. If you don't know how, you can try to change getApplicationInfo like this so that it will send a periodic error :

let index = 0;
export const getApplicationInfo = () => {
  index += 1;
  if (index === 10) {
    index = 0;
    throw FunctionalError('for debug');
  }
  return {
    version: PLATFORM_VERSION,
    debugStats: {}, // Lazy loaded
  };
};

@SouadHadjiat SouadHadjiat added the filigran team use to identify PR from the Filigran team label Jan 24, 2025
@lndrtrbn lndrtrbn self-requested a review January 27, 2025 09:06
@SouadHadjiat SouadHadjiat merged commit af750f2 into master Jan 27, 2025
5 of 6 checks passed
@SouadHadjiat SouadHadjiat deleted the opencti/issue/9693 branch January 27, 2025 10:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

filigran team use to identify PR from the Filigran team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants