Prometheus exporter plugin, found security issues

Hi,

when using Search Guard with 3rd party Elasticsearch plugin I am hitting issues that might represent security risk. At this point I am not sure if the issue is cause by SG or the 3rd party plugin. But the plugin is quite simple. Details including code to reproduce the issue follows (sorry for long email…):

Details:

···

===========

I am using Prometheus exporter plugin [1], this plugin introduces new REST API to the ES node. I am aware of the fact that this plugin is not officially listed in supported plugins [2] but first I think it might be very nice to add it there (and I can volunteer to do necessary PRs if needed), second the plugin code is so simple that I am afraid many other plugins can suffer from similar security issues.

I wanted to configure SG to control access to the Prometheus exporter REST API which is exposed at /_prometheus/metrics on each ES node that has this plugin. After looking into plugin code (examples how plugin makes request to ES [3a, 3b]) and doing some experiments I realized that SG will require client accessing ES node to have the following two permissions:

  • cluster:metrics/health

  • cluster:metrics/nodes/stats

(- and in never versions also indices stats… but leave this out for now)

So far so good. I created new user that has required permissions and as expected it can pull prometheus metrics from ES node. However, I also discovered that users that do not have required permissions can pull metrics as well. Or opposite, i.e. clients with appropriate permissions are rejected. As the following code example demonstrates access to Prometheus REST API can (or can not) work as expected depending which user makes call to the REST API before them (but it is probably more complicated).

Here is the example code:

===========

Using SG 2.4.4.x, ES 2.4.4.x

Imagine I define three SG internal users: ‘first_perm’, ‘second_perm’ and ‘both_perms’. The only user that should be always allowed to pull from prometheus REST API is ‘both_perms’. The other two users always miss first or send permission from the two needed.

In the end I created simple test script that demonstrate two different access cases.

Case 1)

  • user ‘first_perm’ call the API first

  • then ‘second_perm’ user

  • last ‘both_perms’

https://github.com/lukas-vlcek/es-sg-prometheus-integration/blob/case_1/test.sh

Case 2)

the order of users is reversed

  • first comes ‘both_perms’

  • then ‘second_perms’

  • last ‘first_perm’

https://github.com/lukas-vlcek/es-sg-prometheus-integration/blob/case_2/test.sh

As you can see apart from call to prometheus REST API each user is also calling “raw” cluster health and nodes stats to demonstrate if it can access it as well.

As you can see the only difference between code base in both cases is the order:

https://github.com/lukas-vlcek/es-sg-prometheus-integration/compare/case_1…case_2

Surprisingly, the output of both cases is dramatically different.

What I found broken (and what I consider security risk):

===========

First of all, complete logs can be found here:

case_1:

  • output of test.sh script [4] (starting at line 2205)

  • Elasticsearch log [5] (starting at line 2242)

case_2:

  • output of test.sh script [6] (starting at line 2203)

  • Elasticsearch log [7] (starting at line 2255)

For case 1:

Both ‘first_perm’ and ‘second_perm’ users are rejected access to prometheus REST API (that is expected) and both can access one of the cluster/health or nodes/stats accordingly. But the ‘both_perms’ user is not allowed to hit prometheus REST API (while at the same time it can hit both cluster/health and modes/stats individually). Further inspection in ES log reveals interesting detail. When ‘both_perms’ makes request to Prometheus API we can see [8] it is pulled from internal backend (looks ok) and evaluated permissions are for user ‘second_perm’ (!!!) user, thus rejected…

For case 2:

This time ‘both_perms’ user comes first and it is allowed to access prometheus REST API (yea, cool!). Then ‘second_perm’ user comes and it is also allowed to access it (!!!, it shouldn’t). The same apply to ‘first_perm’ user too (!!!). Further inspection of ES log shows that ‘first_perm’ user is evaluated for user ‘both_perms’ [9] as ‘second_perm’ user is evaluated to ‘both_perms’ [10].

Complete code examples are found in https://github.com/lukas-vlcek/es-sg-prometheus-integration in branches ‘case_1’ and ‘case_2’. For each commit there is Travis service run the code (download ES, install and configure SG, Prometheus plugin and run test.sh). Then log can be fully investigated on Travis as well. Also this can be run locally as well (requires Docker).

My main questions:

===========

Is this issue with SG or Prometheus plugin?

Am I doing anything wrong in my code?

HTH

[1] https://github.com/vvanholl/elasticsearch-prometheus-exporter

[2] http://floragunncom.github.io/search-guard-docs/compatibility.html

[3a] https://github.com/vvanholl/elasticsearch-prometheus-exporter/blob/2.4.4.0/src/main/java/org/compuscene/metrics/prometheus/PrometheusMetricsCollector.java#L519

[3b] https://github.com/vvanholl/elasticsearch-prometheus-exporter/blob/2.4.4.0/src/main/java/org/compuscene/metrics/prometheus/PrometheusMetricsCollector.java#L524

[4] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279171#L2205

[5] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279171#L2242

[6] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2203

[7] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2255

[8] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279171#L2488-L2493

[9] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2514-L2519

[10] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2461-L2466

Regards,

Lukáš

This is an issue with prometheus plugin: thread context handling is buggy · Issue #57 · vvanholl/elasticsearch-prometheus-exporter · GitHub

···

On Friday, 29 September 2017 17:36:43 UTC+2, Lukáš Vlček wrote:

Hi,

when using Search Guard with 3rd party Elasticsearch plugin I am hitting issues that might represent security risk. At this point I am not sure if the issue is cause by SG or the 3rd party plugin. But the plugin is quite simple. Details including code to reproduce the issue follows (sorry for long email…):

Details:

===========

I am using Prometheus exporter plugin [1], this plugin introduces new REST API to the ES node. I am aware of the fact that this plugin is not officially listed in supported plugins [2] but first I think it might be very nice to add it there (and I can volunteer to do necessary PRs if needed), second the plugin code is so simple that I am afraid many other plugins can suffer from similar security issues.

I wanted to configure SG to control access to the Prometheus exporter REST API which is exposed at /_prometheus/metrics on each ES node that has this plugin. After looking into plugin code (examples how plugin makes request to ES [3a, 3b]) and doing some experiments I realized that SG will require client accessing ES node to have the following two permissions:

  • cluster:metrics/health
  • cluster:metrics/nodes/stats

(- and in never versions also indices stats… but leave this out for now)

So far so good. I created new user that has required permissions and as expected it can pull prometheus metrics from ES node. However, I also discovered that users that do not have required permissions can pull metrics as well. Or opposite, i.e. clients with appropriate permissions are rejected. As the following code example demonstrates access to Prometheus REST API can (or can not) work as expected depending which user makes call to the REST API before them (but it is probably more complicated).

Here is the example code:

===========

Using SG 2.4.4.x, ES 2.4.4.x

Imagine I define three SG internal users: ‘first_perm’, ‘second_perm’ and ‘both_perms’. The only user that should be always allowed to pull from prometheus REST API is ‘both_perms’. The other two users always miss first or send permission from the two needed.

In the end I created simple test script that demonstrate two different access cases.

Case 1)

  • user ‘first_perm’ call the API first
  • then ‘second_perm’ user
  • last ‘both_perms’

https://github.com/lukas-vlcek/es-sg-prometheus-integration/blob/case_1/test.sh

Case 2)

the order of users is reversed

  • first comes ‘both_perms’
  • then ‘second_perms’
  • last ‘first_perm’

https://github.com/lukas-vlcek/es-sg-prometheus-integration/blob/case_2/test.sh

As you can see apart from call to prometheus REST API each user is also calling “raw” cluster health and nodes stats to demonstrate if it can access it as well.

As you can see the only difference between code base in both cases is the order:

https://github.com/lukas-vlcek/es-sg-prometheus-integration/compare/case_1…case_2

Surprisingly, the output of both cases is dramatically different.

What I found broken (and what I consider security risk):

===========

First of all, complete logs can be found here:

case_1:

  • output of test.sh script [4] (starting at line 2205)
  • Elasticsearch log [5] (starting at line 2242)

case_2:

  • output of test.sh script [6] (starting at line 2203)
  • Elasticsearch log [7] (starting at line 2255)

For case 1:

Both ‘first_perm’ and ‘second_perm’ users are rejected access to prometheus REST API (that is expected) and both can access one of the cluster/health or nodes/stats accordingly. But the ‘both_perms’ user is not allowed to hit prometheus REST API (while at the same time it can hit both cluster/health and modes/stats individually). Further inspection in ES log reveals interesting detail. When ‘both_perms’ makes request to Prometheus API we can see [8] it is pulled from internal backend (looks ok) and evaluated permissions are for user ‘second_perm’ (!!!) user, thus rejected…

For case 2:

This time ‘both_perms’ user comes first and it is allowed to access prometheus REST API (yea, cool!). Then ‘second_perm’ user comes and it is also allowed to access it (!!!, it shouldn’t). The same apply to ‘first_perm’ user too (!!!). Further inspection of ES log shows that ‘first_perm’ user is evaluated for user ‘both_perms’ [9] as ‘second_perm’ user is evaluated to ‘both_perms’ [10].

Complete code examples are found in https://github.com/lukas-vlcek/es-sg-prometheus-integration in branches ‘case_1’ and ‘case_2’. For each commit there is Travis service run the code (download ES, install and configure SG, Prometheus plugin and run test.sh). Then log can be fully investigated on Travis as well. Also this can be run locally as well (requires Docker).

My main questions:

===========

Is this issue with SG or Prometheus plugin?

Am I doing anything wrong in my code?

HTH

[1] https://github.com/vvanholl/elasticsearch-prometheus-exporter

[2] http://floragunncom.github.io/search-guard-docs/compatibility.html

[3a] https://github.com/vvanholl/elasticsearch-prometheus-exporter/blob/2.4.4.0/src/main/java/org/compuscene/metrics/prometheus/PrometheusMetricsCollector.java#L519

[3b] https://github.com/vvanholl/elasticsearch-prometheus-exporter/blob/2.4.4.0/src/main/java/org/compuscene/metrics/prometheus/PrometheusMetricsCollector.java#L524

[4] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279171#L2205

[5] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279171#L2242

[6] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2203

[7] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2255

[8] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279171#L2488-L2493

[9] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2514-L2519

[10] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2461-L2466

Regards,

Lukáš

Just in case, there is now PR for Prometheus exporter plugin to fix this: Prometheus exporter fix for v2.x by lukas-vlcek · Pull Request #64 · vvanholl/elasticsearch-prometheus-exporter · GitHub

Regards,

Lukáš

Dne neděle 1. října 2017 23:06:38 UTC+2 Search Guard napsal(a):

···

This is an issue with prometheus plugin: https://github.com/vvanholl/elasticsearch-prometheus-exporter/issues/57

On Friday, 29 September 2017 17:36:43 UTC+2, Lukáš Vlček wrote:

Hi,

when using Search Guard with 3rd party Elasticsearch plugin I am hitting issues that might represent security risk. At this point I am not sure if the issue is cause by SG or the 3rd party plugin. But the plugin is quite simple. Details including code to reproduce the issue follows (sorry for long email…):

Details:

===========

I am using Prometheus exporter plugin [1], this plugin introduces new REST API to the ES node. I am aware of the fact that this plugin is not officially listed in supported plugins [2] but first I think it might be very nice to add it there (and I can volunteer to do necessary PRs if needed), second the plugin code is so simple that I am afraid many other plugins can suffer from similar security issues.

I wanted to configure SG to control access to the Prometheus exporter REST API which is exposed at /_prometheus/metrics on each ES node that has this plugin. After looking into plugin code (examples how plugin makes request to ES [3a, 3b]) and doing some experiments I realized that SG will require client accessing ES node to have the following two permissions:

  • cluster:metrics/health
  • cluster:metrics/nodes/stats

(- and in never versions also indices stats… but leave this out for now)

So far so good. I created new user that has required permissions and as expected it can pull prometheus metrics from ES node. However, I also discovered that users that do not have required permissions can pull metrics as well. Or opposite, i.e. clients with appropriate permissions are rejected. As the following code example demonstrates access to Prometheus REST API can (or can not) work as expected depending which user makes call to the REST API before them (but it is probably more complicated).

Here is the example code:

===========

Using SG 2.4.4.x, ES 2.4.4.x

Imagine I define three SG internal users: ‘first_perm’, ‘second_perm’ and ‘both_perms’. The only user that should be always allowed to pull from prometheus REST API is ‘both_perms’. The other two users always miss first or send permission from the two needed.

In the end I created simple test script that demonstrate two different access cases.

Case 1)

  • user ‘first_perm’ call the API first
  • then ‘second_perm’ user
  • last ‘both_perms’

https://github.com/lukas-vlcek/es-sg-prometheus-integration/blob/case_1/test.sh

Case 2)

the order of users is reversed

  • first comes ‘both_perms’
  • then ‘second_perms’
  • last ‘first_perm’

https://github.com/lukas-vlcek/es-sg-prometheus-integration/blob/case_2/test.sh

As you can see apart from call to prometheus REST API each user is also calling “raw” cluster health and nodes stats to demonstrate if it can access it as well.

As you can see the only difference between code base in both cases is the order:

https://github.com/lukas-vlcek/es-sg-prometheus-integration/compare/case_1…case_2

Surprisingly, the output of both cases is dramatically different.

What I found broken (and what I consider security risk):

===========

First of all, complete logs can be found here:

case_1:

  • output of test.sh script [4] (starting at line 2205)
  • Elasticsearch log [5] (starting at line 2242)

case_2:

  • output of test.sh script [6] (starting at line 2203)
  • Elasticsearch log [7] (starting at line 2255)

For case 1:

Both ‘first_perm’ and ‘second_perm’ users are rejected access to prometheus REST API (that is expected) and both can access one of the cluster/health or nodes/stats accordingly. But the ‘both_perms’ user is not allowed to hit prometheus REST API (while at the same time it can hit both cluster/health and modes/stats individually). Further inspection in ES log reveals interesting detail. When ‘both_perms’ makes request to Prometheus API we can see [8] it is pulled from internal backend (looks ok) and evaluated permissions are for user ‘second_perm’ (!!!) user, thus rejected…

For case 2:

This time ‘both_perms’ user comes first and it is allowed to access prometheus REST API (yea, cool!). Then ‘second_perm’ user comes and it is also allowed to access it (!!!, it shouldn’t). The same apply to ‘first_perm’ user too (!!!). Further inspection of ES log shows that ‘first_perm’ user is evaluated for user ‘both_perms’ [9] as ‘second_perm’ user is evaluated to ‘both_perms’ [10].

Complete code examples are found in https://github.com/lukas-vlcek/es-sg-prometheus-integration in branches ‘case_1’ and ‘case_2’. For each commit there is Travis service run the code (download ES, install and configure SG, Prometheus plugin and run test.sh). Then log can be fully investigated on Travis as well. Also this can be run locally as well (requires Docker).

My main questions:

===========

Is this issue with SG or Prometheus plugin?

Am I doing anything wrong in my code?

HTH

[1] https://github.com/vvanholl/elasticsearch-prometheus-exporter

[2] http://floragunncom.github.io/search-guard-docs/compatibility.html

[3a] https://github.com/vvanholl/elasticsearch-prometheus-exporter/blob/2.4.4.0/src/main/java/org/compuscene/metrics/prometheus/PrometheusMetricsCollector.java#L519

[3b] https://github.com/vvanholl/elasticsearch-prometheus-exporter/blob/2.4.4.0/src/main/java/org/compuscene/metrics/prometheus/PrometheusMetricsCollector.java#L524

[4] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279171#L2205

[5] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279171#L2242

[6] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2203

[7] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2255

[8] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279171#L2488-L2493

[9] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2514-L2519

[10] https://travis-ci.org/lukas-vlcek/es-sg-prometheus-integration/builds/281279030#L2461-L2466

Regards,

Lukáš