Error pulling events when using MySQL for CCDB

I'm getting the following error when moving to a newer version of AdminUI that pulls events:
D, [2015-06-17T09:32:22.982008 #2480] DEBUG -- : [ -- ] : [ -- ] : Select for key events, table events: SELECT `actee`, `actee_name`, `actee_type`, `actor`, `actor_name`, `actor_type`, `created_at`, `guid`, `id`, `metadata`, `organization_guid`, `space_guid`, `space_id`, `timestamp`, `type`, `updated_at` FROM `events` WHERE ("timestamp" >= CURRENT_TIMESTAMP - INTERVAL '7 DAYS')
D, [2015-06-17T09:32:22.982061 #2480] DEBUG -- : [ -- ] : [ -- ] : Columns removed for key events, table events: []
D, [2015-06-17T09:32:22.983412 #2480] DEBUG -- : [ -- ] : [ -- ] : Error during discovery of events: #<Sequel::DatabaseError: Mysql2::Error: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ')' at line 1>
D, [2015-06-17T09:32:22.983513 #2480] DEBUG -- : [ -- ] : [ -- ] : /usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/adapters/mysql2.rb:78:in `query'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/adapters/mysql2.rb:78:in `block in _execute'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/database/logging.rb:33:in `log_yield'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/adapters/mysql2.rb:78:in `_execute'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/adapters/shared/mysql_prepared_statements.rb:34:in `block in execute'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/database/connecting.rb:250:in `block in synchronize'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/connection_pool/threaded.rb:103:in `hold'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/database/connecting.rb:250:in `synchronize'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/adapters/shared/mysql_prepared_statements.rb:34:in `execute'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/dataset/actions.rb:921:in `execute'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/adapters/mysql2.rb:195:in `execute'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/adapters/mysql2.rb:154:in `fetch_rows'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/dataset/actions.rb:139:in `each'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/database/dataset.rb:46:in `fetch'
/home/adminui/app/admin-ui/lib/admin/cc.rb:640:in `block in select'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/database/connecting.rb:69:in `connect'
/usr/local/rvm/gems/ruby-1.9.3-p551/gems/sequel-4.23.0/lib/sequel/core.rb:95:in `connect'
/home/adminui/app/admin-ui/lib/admin/cc.rb:610:in `select'
/home/adminui/app/admin-ui/lib/admin/cc.rb:550:in `discover'
/home/adminui/app/admin-ui/lib/admin/cc.rb:578:in `block in schedule'
/home/adminui/app/admin-ui/lib/admin/scheduled_thread_pool.rb:27:in `call'
/home/adminui/app/admin-ui/lib/admin/scheduled_thread_pool.rb:27:in `block (2 levels) in initialize'

It looks like the SQL syntax being generated is invalid for MySQL.
Aaron Huber
Intel Corporation

@amhuber, I determined a fix and will commit a change tomorrow to address.

@amhuber, please try commit https://github.com/cloudfoundry-incubator/admin-ui/commit/f18e2465f1fef38dd110cd684123611951c75582

Well, that did fix the error but it still doesn't seem to be working.  Now it just says there are 0 events:
D, [2015-06-18T10:08:51.124115 #7192] DEBUG -- : [ -- ] : [ -- ] : [150 second interval] Starting view model events discovery...
D, [2015-06-18T10:08:51.124174 #7192] DEBUG -- : [ -- ] : [ -- ] : Caching view model events data.  Compilation time: 1.5474e-05 seconds
D, [2015-06-18T10:08:51.199199 #7192] DEBUG -- : [ -- ] : [ -- ] : [300 second interval] Starting CC events discovery...
D, [2015-06-18T10:08:51.228147 #7192] DEBUG -- : [ -- ] : [ -- ] : Select for key events, table events: SELECT `actee`, `actee_name`, `actee_type`, `actor`, `actor_name`, `actor_type`, `created_at`, `guid`, `id`, `metadata`, `organization_guid`, `space_guid`, `space_id`, `timestamp`, `type`, `updated_at` FROM `events` WHERE "timestamp" >= CURRENT_TIMESTAMP - INTERVAL '7' DAY
D, [2015-06-18T10:08:51.228221 #7192] DEBUG -- : [ -- ] : [ -- ] : Columns removed for key events, table events: []
D, [2015-06-18T10:08:51.230138 #7192] DEBUG -- : [ -- ] : [ -- ] : Caching CC events data.  Count: 0.  Retrieval time: 0.030858964 seconds
D, [2015-06-18T10:09:01.148820 #7192] DEBUG -- : [ -- ] : [ -- ] : [150 second interval] Starting view model events discovery...
D, [2015-06-18T10:09:01.148918 #7192] DEBUG -- : [ -- ] : [ -- ] : Caching view model events data.  Compilation time: 4.0182e-05 seconds
D, [2015-06-18T10:11:31.199176 #7192] DEBUG -- : [ -- ] : [ -- ] : [150 second interval] Starting view model events discovery...
D, [2015-06-18T10:11:31.199269 #7192] DEBUG -- : [ -- ] : [ -- ] : Caching view model events data.  Compilation time: 4.9348e-05 seconds
D, [2015-06-18T10:13:51.540473 #7192] DEBUG -- : [ -- ] : [ -- ] : [300 second interval] Starting CC events discovery...
D, [2015-06-18T10:13:51.559106 #7192] DEBUG -- : [ -- ] : [ -- ] : Caching CC events data.  Count: 0.  Retrieval time: 0.018569781 seconds
D, [2015-06-18T10:14:01.234742 #7192] DEBUG -- : [ -- ] : [ -- ] : [150 second interval] Starting view model events discovery...
D, [2015-06-18T10:14:01.234840 #7192] DEBUG -- : [ -- ] : [ -- ] : Caching view model events data.  Compilation time: 5.3178e-05 seconds
D, [2015-06-18T10:16:31.272851 #7192] DEBUG -- : [ -- ] : [ -- ] : [150 second interval] Starting view model events discovery...
D, [2015-06-18T10:16:31.272993 #7192] DEBUG -- : [ -- ] : [ -- ] : Caching view model events data.  Compilation time: 5.3608e-05 seconds

I tested the query manually from the mysql client and I do get records:
mysql> set session sql_mode = 'ANSI';
Query OK, 0 rows affected (0.01 sec)

SELECT `actee`, `actee_name`, `actee_type`, `actor`, `actor_name`, `actor_type`, `created_at`, `guid`, `id`, `metadata`, `organization_guid`, `space_guid`, `space_id`, `timestamp`, `type`, `updated_at` FROM `events` WHERE "timestamp" >= CURRENT_TIMESTAMP - INTERVAL '7' DAY;
<data>
2178 rows in set (0.10 sec)

I'll poke through the code to see if I can find anything.
Aaron

Here is one row of data in case that helps:
mysql> SELECT `actee`, `actee_name`, `actee_type`, `actor`, `actor_name`, `actor_type`, `created_at`, `guid`, `id`, `metadata`, `organization_guid`, `space_guid`, `space_id`, `timestamp`, `type`, `updated_at` FROM `events` WHERE "timestamp" >= CURRENT_TIMESTAMP - INTERVAL '7' DAY LIMIT 1;
+--------------------------------------+-------------+------------+--------------------------------------+------------+------------+---------------------+--------------------------------------+----+-----------------------------------------------------------------------------------------------+--------------------------------------+--------------------------------------+----------+---------------------+--------------------+------------+
| actee                                | actee_name  | actee_type | actor                                | actor_name | actor_type | created_at          | guid                                 | id | metadata                                                                                | organization_guid                    | space_guid                           | space_id | timestamp           | type               | updated_at |
+--------------------------------------+-------------+------------+--------------------------------------+------------+------------+---------------------+--------------------------------------+----+-----------------------------------------------------------------------------------------------+--------------------------------------+--------------------------------------+----------+---------------------+--------------------+------------+
| 60e26603-661a-4957-8223-ece0d17a9e0c | Development | space      | 032ec221-8eb3-4652-a021-7be438c2732e | admin      | user       | 2015-06-12 11:25:36 | 488c7e58-fda7-4464-9538-8bf683ce9a18 |  2 | {"request":{"name":"Development","organization_guid":"8a84b780-c82f-4968-8a79-62cfa389f85f"}} | 8a84b780-c82f-4968-8a79-62cfa389f85f | 60e26603-661a-4957-8223-ece0d17a9e0c |        2 | 2015-06-12 18:25:36 | audit.space.create | NULL       |
+--------------------------------------+-------------+------------+--------------------------------------+------------+------------+---------------------+--------------------------------------+----+-----------------------------------------------------------------------------------------------+--------------------------------------+--------------------------------------+----------+---------------------+--------------------+------------+
1 row in set (0.00 sec)

Aaron

Just did some testing with the Sequel gem, looks like this is just due to the quoting around the field name in the where clause:
require 'sequel'

items = []

Sequel.connect('mysql2://<snip>') do |connection|
  #Doesn't work
  #connection.fetch("SELECT `actee`, `actee_name`, `actee_type`, `actor`, `actor_name`, `actor_type`, `created_at`, `guid`, `id`, `metadata`, `organization_guid`, `space_guid`, `space_id`, `timestamp`, `type`, `updated_at` FROM `events` WHERE \"timestamp\" >= CURRENT_TIMESTAMP - INTERVAL '1' DAY") do |row|

  #Works
  connection.fetch("SELECT `actee`, `actee_name`, `actee_type`, `actor`, `actor_name`, `actor_type`, `created_at`, `guid`, `id`, `metadata`, `organization_guid`, `space_guid`, `space_id`, `timestamp`, `type`, `updated_at` FROM `events` WHERE `timestamp` >= CURRENT_TIMESTAMP - INTERVAL '1' DAY") do |row|
    items.push(row)
  end
end

print items.count

I also tried a single quote and it didn't work either.  It worked with either no quotes or with the back-tick character.
Bad:
\"timestamp\"
'timestamp'

Good:
`timestamp`
timestamp

Now hopefully you can find one that works for Postgres as well.  :-)
Aaron

Just a final confirmation, everything works as expected when I used backticks in the code:
where:   "`timestamp` >= CURRENT_TIMESTAMP - INTERVAL<EMAIL_ADDRESS>DAY"

Log entry:
D, [2015-06-18T11:07:23.809563 #11193] DEBUG -- : [ -- ] : [ -- ] : Caching CC events data.  Count: 2178.  Retrieval time: 0.150513388 seconds

Page view looks good, etc.
Aaron

@amhuber, back tick does not work with Postgres.  But, no quote around timestamp appears to work.  I'll run through the rspec tests and do some more freelance testing to make sure.

Taking the back ticks out works fine on MySQL as well, just did a full test:
D, [2015-06-18T11:26:27.540424 #13120] DEBUG -- : [ -- ] : [ -- ] : Caching CC events data.  Count: 2178.  Retrieval time: 0.261141404 seconds

@amhuber, thanks for the legwork.  Here is a commit to remove the quotes from around timestamp in the events predicate:  https://github.com/cloudfoundry-incubator/admin-ui/commit/9802863167dadb22d294b37c1722f9467b8cb5fb

Confirmed working, thanks!


link to data supporting masks from surveys and website

It would be neat if there was some way to encourage people to follow prevention measures after they interact with the website. This could involve follow-up to surveys recommending the behavior, showing data to persuade for the behavior, or maybe making a comical follow-up like an animated gif that promotes mask wearing/photo of local icon like Dippy wearing a mask.
User story
When I see people outside it seems random whether they wear a mask or not, and it would be nice to encourage mask wearing more.
Additional context

I considered adding a "learn more about the efficacy of wearing masks", link but I think we should wait until we have published our study on causal impact.

wontfix


I am facing the issue of the 'self sign certificate in certification chain', while using the main sender tool (nodemailer) in my express framework, please help me out

Please do not submit this issue.

[X] I understand

### Tasks

Error: self-signed certificate in certificate chain at TLSSocket.onConnectSecure (node:_tls_wrap:1674:34) at TLSSocket.emit (node:events:518:28) at TLSSocket._finishInit (node:_tls_wrap:1085:8) at ssl.onhandshakedone (node:_tls_wrap:871:12) { code: 'ESOCKET', command: 'CONN' }

This issue is not following the bug template. Please re-file and provide a self-contained reproduction, which we can run locally to reproduce your issue. Thanks for your understanding!


11193 DEV update dashboard/dashboardNew to able to edit supportEmail if technicalSupportAvailable is true

https://github.com/Shippable/pm/issues/11193

Verified on RC.


[pwm, dv] Add pwm_smoke test for PWM IP

Currently, the pwm_smoke test verifies only the Standard operation mode of the PWM IP
Refer PR #7093
Signed-off-by: Tung Hoang<EMAIL_ADDRESS>

@weicaiyang I have updated this PR to address all your comments, please do the second round of review if required


Neural network back propagation p bblas

The implementation of NeuralNetworks by using PBblas library

This is an effective use of the PBblas attributes.

I am implementing Stacked Sparse Autoencoder for which I need all my previous implementations. So I have merged all my implementations by now in a new branch that includes NeuralNetwork, DeepLearning and SoftMax implementations. I will make a pull request for that branch when my implementation of Stacked Autoencoder is completed.


New workflow submission - Moderated Image Catalog pattern using CDK with Python

To submit a workflow to the Step Functions Workflows Collection, submit an issue with the following information.
To learn more about submitting a workflow, read the publishing guidelines page.
Note the following information for the workflow:

Name: Moderated Image Catalog
Description: This workflow implements a moderated image cataloging pipeline.
Simplicity: 3 - Application
Diagram: Included
Type: Standard
Resources should link to AWS documentation and AWS blogs related to the post (1-5 maximum).
Framework: CDK
Language: Python
PR: #152
Original Author of workflow: Pascal Vogel, Benjamin Meyer
CDK Author: Adrian Hunter

You must ensure that the sections of the model README.md are completed in full.

merged


APD-14039: Handle primary keys order

https://nexttrucking.atlassian.net/browse/APD-14039
the issue happens when update/delete enabled, because kafka key is being used when doing update/delete
https://github.com/NextDeveloperTeam/kafka-connect-bigquery/blob/master/kcbq-connector/src/main/java/com/wepay/kafka/connect/bigquery/MergeQueries.java#L227
if the kafka key is a record (means it has sub fields, means multiple primary keys from the original table), the order of those keys are not consistent, if the tmp table (containing the update/delete records)'s key order is different with the destination table, so when merge the tmp table with destination table, the above line of code will break because the kafka key is changed
In order to solve this, when creating both destination/tmp table, we should fix the order of kafka key if they have multiple fields, currently the order just sorted based on the field name, and this might not reflect the real primary key order from the original table, so to minimize the impact, we just set this as a flag, when such things happens, we just need to turn on the flag to resolve this
The root cause might be from the schema retriever, which is not guaranteeing the order of primary keys
curl --silent -X GET http://schema-registry-cp-schema-registry.kafka.svc.cluster.local:8081/subjects/debezium_nextgeo-001.public.gps_load-key/versions/latest | jq .
{
"subject": "debezium_nextgeo-001.public.gps_load-key",
"version": 2,
"id": 650,
"schema": "{"type":"record","name":"Key","namespace":"debezium_nextgeo_001.public.gps_load","fields":[{"name":"user_id","type":"string"},{"name":"sequence_id","type":"long"},{"name":"load_id","type":"string"}],"connect.name":"debezium_nextgeo_001.public.gps_load.Key"}"
}

the order of those keys are not consistent

Do we know why this happens?  Can we fix further upstream so that we fix it in one place (debezium?) and in each connector sink?
Another thought - is this fixing it one in the scope of the temp table or is this tableschema used elsewhere in the code?  If it's not scoped to just the temp table code, would it be possible to move this logic there (or instead just use the order defined on the actual table when building the temp table, ignoring whatever is passed in with this definition)?
Hope this makes some sense... I'd jump on a call but my internet is down and mobile coverage is slowwwwwww at home.

Do we know why this happens? Can we fix further upstream so that we fix it in one place (debezium?) and in each connector sink?
schema is built on the record (the last record in a batch which by default contains 500 records) for both tmp and destination tables, for some reason, the record starts to change the order of the key since 12/2020. So when it builds the original/tmp tables, if the new data (with the changed order) coming syned directly to the original tmp table, it is still fine, because the tmp table's schema is already determined by the old record. However, if the original tmp table disappear(or we kill the connector). Then when new records coming, it will build another tmp table which will have the new order key, when it merges with the destination table using kafka key, it fails

so that's why I doubt this is data change (but we didn't find any clue when the data got changed)

Another thought - is this fixing it one in the scope of the temp table or is this tableschema used elsewhere in the code? If it's not scoped to just the temp table code, would it be possible to move this logic there (or instead just use the order defined on the actual table when building the temp table, ignoring whatever is passed in with this definition)?

this fix is apply to both final table and all tmp tables!


Allow billing address to be sent with token.

Allows the billing address to be sent to Vantiv/Litle along with the string token. Also adds in the name and email address.

Are we going to try to get this pulled in upstream?  If so, we might need to expand the commit message to clarify why this is necessary.  Perhaps a pointer to documentation that shows addresses can/should be sent alongside tokens?


updated xunit commands for new runner

This switches xunit arguments to be compatible with the new aspnet.xunit.runner.
--test becomes -method
I've also added a fixture -class command that we had in v1.
If you attempt to run a single test when the cursor is at the class level, then the fixture is ran instead.

This seems to be working for me now with dnx projects. I tested it with the KRuntime solution and was able to successfully run single tests, fixtures or assemblies.
The command it generates looks like this
/Users/jason/.dnx/runtimes/dnx-mono.1.0.0-beta5-11427/bin/dnx . test -method Microsoft.Framework.Runtime.Tests.ServiceProviderFacts.NonRegisteredServicesCanBeIEnumerableResolved

OK....  I need to come back to this when I have some more time.


Zuul upgrade to iam-2.0.1, Decode secure metadata field

iam-user-scope-check is Deprecated
Encryption and Decryption Metadata field

Sync  parameters IAM and Zuul.

IAM
iam:
  jwt:
    key: my-jwt-encrypt-key-my-jwt-encrypt-key
    metadata-encoder-secret1: Bar12345Bar12345
    metadata-encoder-secret2: ThisIsASecretKet
    secure-metadata-fields:
      - "$.gitlabToken"

Zuul
zuul:
  iam-jwt-key: my-jwt-encrypt-key-my-jwt-encrypt-key
  metadata-encoder-secret1: Bar12345Bar12345
  metadata-encoder-secret2: ThisIsASecretKet
  secure-metadata-fields:
    - "$.gitlabToken"

iam-jwt-key is JWT signature key. It is used to check whether the token is forged or not.
metadata-encoder-secret1,2 is used to encrypt certain metadata fields in the AES128 encryption format to prevent exposure to JWT tokens.
secure-metadata-fields is a list of metadata fields to be decrypted, expressed in JsonPath syntax.

** metadata JsonPath Examples**
Given the json
{
    "store": {
        "book": [
            {
                "category": "reference",
                "author": "Nigel Rees",
                "title": "Sayings of the Century",
                "price": 8.95
            },
            {
                "category": "fiction",
                "author": "Evelyn Waugh",
                "title": "Sword of Honour",
                "price": 12.99
            },
            {
                "category": "fiction",
                "author": "Herman Melville",
                "title": "Moby Dick",
                "isbn": "0-553-21311-3",
                "price": 8.99
            },
            {
                "category": "fiction",
                "author": "J. R. R. Tolkien",
                "title": "The Lord of the Rings",
                "isbn": "0-395-19395-8",
                "price": 22.99
            }
        ],
        "bicycle": {
            "color": "red",
            "price": 19.95
        }
    },
    "expensive": 10
}

JsonPath (click link to try)
Result

$.store.book[*].author
The authors of all books

$..author
All authors

$.store.*
All things, both books and bicycles

$.store..price
The price of everything

$..book[2]
The third book

$..book[-2]
The second to last book

$..book[0,1]
The first two books

$..book[:2]
All books from index 0 (inclusive) until index 2 (exclusive)

$..book[1:2]
All books from index 1 (inclusive) until index 2 (exclusive)

$..book[-2:]
Last two books

$..book[2:]
Book number two from tail

$..book[?(@.isbn)]
All books with an ISBN number

<EMAIL_ADDRESS>< 10)]
All books in store cheaper than 10

<EMAIL_ADDRESS><= $['expensive'])]
All books in store that are not "expensive"

<EMAIL_ADDRESS>=~ /.*REES/i)]
All books matching regex (ignore case)

$..*
Give me every thing

$..book.length()
The number of books

Working Example

There is an user.

After logging in, IAM return the user's token which is encrypted with the value of gitlabToken.

Let's see zuul router configuration.

zuul:
  metadata-encoder-secret1: Bar12345Bar12345
  metadata-encoder-secret2: ThisIsASecretKet
  secure-metadata-fields:
    - "$.gitlabToken"

  routes:
    gitlab:
      url: http://gitlab.pas-mini.io/api/v4/
      stripPrefix: true
      addHeader:
        PHONE: "010-0000-0000"
        EMAIL: "$.email"
        PRIVATE-TOKEN: "$.gitlabToken"

The above configuration will be sent with the following header, with proxy communication for the /gitlab pass.

PHONE is not starts with $, it will use just string 010-0000-0000
EMAIL is starts with $, it will use metadata email value<EMAIL_ADDRESS>PRIVATE-TOKEN is already listed in secure-metadata-fields, so it will use
decoded value of gitlabToken, 2A_LokKTbnywjw8shHy-

User Scope Tutorial
Step1. Download iam-api-2.0.1.js
In your front-end web, replace static/iam-client/iam-*.js to static/iam-client/iam-api-2.0.1.js
Also replace text In index.html

Step2. IAM application.yml
Your IAM has cloud-server and bpm scopes basically.
scopes:
  - name: cloud-server
    description: 클라우드 서버 접근을 허용합니다.
  - name: bpm
    description: bpm 서비스를 사용합니다.

If your client need both cloud-server and bpm scopes to work well,
You must fill in enable-scopes as follows:
clients:
  - name: uEngine-cloud
    description: uEngine cloud Application
    clientKey: my-client-key
    clientSecret: my-client-secret

    # 토큰 발급시 요청할 수 있는 스코프 목록.
    enable-scopes: cloud-server,bpm

Step3. Edit User's scope
After IAM booted,  go to http://iam.pas-mini.io. Default admin login id-password is admin,admin
Create or modify user.

Let's just check the 'cloud-server' scope on the screen.
Step4. Login from front-web
At router/index.js , add router with scopes as follows:
{
      path: '/auth/:command',
      name: 'login',
      component: Login,
      props: {
        iamServer: iamUrl,
        scopes: "cloud-server,bpm"
      },
      beforeEnter: RouterGuard.requireGuest
    }

Run.
npm install
npm run dev

Your login page looks like:

By default, users can log in even if they only have a cloud-server scope. If you want users to be able to log in with all requested scopes, you must set the following in IAM application.yml:
clients:
  - name: uEngine-cloud
    ....
    user-scope-check-all: true

Step5. Check token
After login, copy access_token from browser tool.

Go to https://jwt.io/ and paste token.

There is only cloud-server scope, Because User has requested two scopes, but the user has only one scope.
Step6. Use token v-if
Iam javascript client has scopes and user, you can use it as follows:

             {{iam.user.userName}} has cloud-server scope.

            {{iam.user.userName}} has bpm scope.

includes -> has 로 바꿔주세욤.. 짧은기 좋아....


[ci] remove comments in CI config

This PR proposes removing repeated comments in the project's CI config which are unrelated to the project.
Changes

Removes Specify service dependencies here comments

Testing
None
Notes
It seems that these comments are not related to hamilton, and might have been from some template config. I don't believe it's necessary to keep them in the project's CI config, and that removing them will reduce the effort necessary to understand that config.
Checklist

[x] PR has an informative and human-readable title (this will be pulled into the release notes)
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
[ ] Passes all existing automated tests
[x] Any change in functionality is tested
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged / future TODOs are captured in comments
[x] Project documentation has been updated if adding/changing functionality.
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Python - local testing

[ ] python 3.6
[ ] python 3.7

Yep, agreed. Artifacts copied over from Stitch Fix internal templates I think.


Remove random ACM logo on the home page banner

If you mean the one below the "Register now!" button, it's actually part of the background image somehow. We'll need to find the original version of that image. (Just commenting this here cause I don't have time to do that right now, but maybe someone else can).

@tasmie / @mghassem - do you have a copy of this image without the ACM logo?

@tnaumann  https://upload.wikimedia.org/wikipedia/commons/d/dc/Shiny_buildings_in_Toronto.jpg


Fix typo

Github -> GitHub

@changeworld : Thanks for your contribution! The author, @davidmu1, has been notified to review your proposed change.


Revert "Added flatlist to tags"

If a FlatList exists, this library cannot be used in ScrollView.
Considering the versatility, it is better not to use FlatList.
This reverts commit 8e90feb8c06deddc7459eedf3e9f16deac45d9da.

iam try it fixed in v10.0.10


Adds time coord checker

Addresses #1770
Adds a utility to check that the time coordinates of a list of cubes match, and that the times are bounded, or not.
Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

I've suggested some minor edits to this PR in #1781.

Superseded by #1781


change base image

Gcc is not pre-installed in python:3-alpha, which will cause the image compilation to fail. So change it to python:3.9.18-slim-bullseye

@feiskyer PTAL


fixes file upload bug

Uploading a file was causing Metis to crash in some browsers (not totally clear on which ones). Maybe a problem with multiparty.form.parse?

https://www.pivotaltracker.com/story/show/165918150

Yeah, I was thinking the same about Clojure when I got to that part.


feat: add ParseOptions field for HertzJWTMiddleware

What type of PR is this?

feat: A new feature
What this PR does / why we need it (English/Chinese):

en: add ParseOptions field for HertzJWTMiddleware
Which issue(s) this PR fixes:

Fixes #12

LGTM


Adds support for transactions with no notes

I was using Actual manualy, so some transactions don't have notes or payees.
I now sync bank accounts, but the older transactions were blocking the actual-ai process.
This PR allows actual-ai to add the transactions without notes to the uncategorizedTransactions array.
I also added various quick fixes:

try/catch block on downloadBudget
Typos in the AI Prompt
NODE_TLS_REJECT_UNAUTHORIZED=0 to README

Thank you!

Great! Thank you!


compilation fails on current formdata

compilation passes if you enable rust-nightly feature.

Maybe #21 fixes this isse?

I'll try this

This should be fixed by #21

confirmed. current head works


RFM - gsmartcontrol

I DON'T Want To Become The Maintainer

[x] I have followed the Package Triage Process and I do NOT want to become maintainer of the package
[x] There is no existing open maintainer request for this package

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details
Package URL: https://community.chocolatey.org/packages/gsmartcontrol
Package source URL: https://github.com/search?q=repo%3Adtgm%2Fchocolatey-packages+filename%3Agsmartcontrol
Date the maintainer was contacted: 2022-06-03 (in another GitHub issue), and September '22 (by email).  No reply.
How the maintainer was contacted:  both via GitHub and via private email
The inactive existing maintainer is @dtgm, who appears to have been inactive for years.
Thank you for reading this!

Hello, I'll update it

Added @tunisiano187 as a maintainer.

Dear @tunisiano187:  I see you've now published a GSmartControl 1.1.4 package.  Thank you very much!
Closing.


fixed text overflow on sidebar item

Before | After

Before
After

Pre-launch Checklist

[x] I have incremented the package version as appropriate and updated CHANGELOG.md with my changes
[x] I have added/updated relevant documentation
[x] I have run "optimize/organize imports" on all changed files
[x] I have addressed all analyzer warnings as best I could

LGTM.


[UI Components] Use ByRole query Dropdown tests

Description
Update the tests of the Dropdown component to use getByRole() in place of getByText().
The trigger button can be queried by the button role:
const dropdownTrigger = screen.getByRole('button', { name: 'button text' });

The menu items can be queried by the menuitem role:
const menu = screen.getByRole('menu')
const menuItem = within(menu).getByRole('menuitem', { name: 'item text' });

File needs to be updated: https://github.com/freeCodeCamp/freeCodeCamp/blob/main/tools/ui-components/src/drop-down/drop-down.test.tsx.

As of I understood by the description of the issue is that you wanted to replace getByRole() to getByText() in the test file of the dropdown component if that's the case I would love to contribute to it


[GHSA-c59h-r6p8-q9wc] Next.js missing cache-control header may lead to CDN caching empty reply

Updates

Affected products

Comments
The problem is that security vulnerability is incorrectly applied to the v0.4 version of next when it was a totally different product, and that affects some packages that still depend on<EMAIL_ADDRESS>Unfortunately, this problem comes back whenever any new vulnerability is added for next.
See https://github.com/github/advisory-database/pull/179
Note: I was unable to setup a valid range in the form (either >=0.9.9  <13.4.20-canary.13 or 0.9.9 - 13.4.20-canary.12), how can we fix it?

Note: I was unable to setup a valid range in the form (either >=0.9.9  <13.4.20-canary.13 or 0.9.9 - 13.4.20-canary.12), how can we fix it?

You were probably missing a , between the lower and upper expressions 😄
Why would 0.9.9 be the correct lower bound though? You're right that 0.4 was completely different but that would make 0.4 the lower bound right?

Closing, as I was able to address it by crafting PR manually here: https://github.com/github/advisory-database/pull/2902


Configure IP Pool in Postal deployed on AWS EC2

Hi All,
I have deployed postal in AWS. I want to set up IP pool for the mail server. I tried assigning new private IPs to the ec2 instance on which postal is running and use those in the IP pool. Is this how we are supposed to configure it because it's not working. My messages are stuck in the queue.
Please help me with the correct guidelines to set this up. Thank you.

Resolved:
Followed steps to add the IP address in network interface mentioned here -> https://aws.amazon.com/premiumsupport/knowledge-center/ec2-ubuntu-secondary-network-interface/


Add proposal for integration testing

We need a proper way to describe, code, and run integration tests. While we have unit tests to verify the correct behavior of functions and methods, we need a way to easily test a network of BigchainDB nodes.
The following proposal address those issues.

Codecov Report

Merging #1949 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1949   +/-   ##
=======================================
  Coverage   98.55%   98.55%
=======================================
  Files          60       60
  Lines        3040     3040
=======================================
  Hits         2996     2996
  Misses         44       44

@ttmc wrote:

I also wonder about tests that aren't integration tests, e.g. performance tests (of things like throughput and latency).

IMO performance tests should be another proposal.


Add links in docstrings

Which issue does this PR close?
N/A
Rationale for this change
While reviewing the docs on https://docs.rs/arrow/5.0.0/arrow/array/trait.Array.html#implementors as part of  https://github.com/apache/arrow-rs/pull/603 (thanks @novemberkilo !)  I found a few structs that are not often used directly but that might not be clear from the docs
Are there any user-facing changes?
Add some doc links to the structs that users might be looking for

Codecov Report

Merging #605 (ea4ad49) into master (6c1a86e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #605   +/-   ##
=======================================
  Coverage   82.41%   82.41%
=======================================
  Files         167      167
  Lines       46240    46240
=======================================
  Hits        38108    38108
  Misses       8132     8132

Impacted Files
Coverage Δ

arrow/src/array/array_binary.rs
92.23% <ø> (ø)

arrow/src/array/array_list.rs
95.51% <ø> (ø)

arrow/src/array/array_string.rs
97.85% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c1a86e...ea4ad49. Read the comment docs.


Report type overloads after doc-string

Is your feature request related to a problem? Please describe.
If a function has type overloads then all overloaded types will be listed first with doc-string at the bottom.
This usually doesn't make much sense, since overload types are usually for better type-checking rather than user-help.
Code example:
from typing import Literal, Union, overload

# fmt: off
@overload
def func(t: Literal["str"]) -> str: ...
@overload
def func(t: Literal["int"]) -> int: ...
# fmt: on

def func(t: str) -> Union[str, int]:
    """
    A description

    :param t: Required type of result
    :return: Some value of requested type
    """
    if t == "str":
        return ""
    if t == "int":
        return 0
    raise ValueError("Unknown type")

Help for the example:

Here with just two overloads it works fine, but if many overloads are used, then only overloads are visible, especially if the function signature itself is long and takes up multiple lines.
Describe the solution you'd like
I would prefer if help was in the following order:
1. Main signature
2. Doc-string
3. All overloaded types
Additional context
There is some uncertainty about what the main signature should be.
In my company we just use Union of all possible types in the main signature, so that should be fine.
Although, I saw that some people leave types in main signature blank, in such cases it might make sense to either try to assemble proper types or leave overloads at the top.

Hmm, I don't use pylance, so my example might not apply there.
I guess I need to raise this with coc team as well.

@Rizhiy, if the pylance team decides to implement this enhancement request or something like it, the behavior will also be reflected in pyright.

Moving this issue to discussion as an enhancement request for comments and upvotes.


Your .github/scripts/close-invalid-link.cjs is broken

Please make sure you have searched for information in the following guides.

[X] Search the issues already opened: https://github.com/GoogleCloudPlatform/google-cloud-node/issues
[X] Check our Troubleshooting guide: https://googlecloudplatform.github.io/google-cloud-node/#/docs/guides/troubleshooting
[X] Check our FAQ: https://googlecloudplatform.github.io/google-cloud-node/#/docs/guides/faq
[X] Check our libraries HOW-TO: https://github.com/googleapis/gax-nodejs/blob/main/client-libraries.md
[X] Check out our authentication guide: https://github.com/googleapis/google-auth-library-nodejs
[X] Check out handwritten samples for many of our APIs: https://github.com/GoogleCloudPlatform/nodejs-docs-samples

Documentation Request
Document this error properly https://github.com/googleapis/google-cloud-node/issues/5824

A link to a public Github Repository or gist with a minimal reproduction.

I have:

A public link
A gist
A minimal reproduction

What do you want?, a frontend to run the SSML having a frontend written by us?

I ran this validation

https://github.com/googleapis/google-cloud-node/blob/main/.github/workflows/issues-no-repro.yaml
https://github.com/googleapis/google-cloud-node/blob/main/.github/scripts/close-invalid-link.cjs

It seems like whoever wrote this code thought, I am getting the markdown, and forever the line that will contain the URL is the 18, it's so wrong, that could be acceptable but if for any reason you have a bug in your code you will close that issue too, you aren't checking the error, and if you fail, you close an issue, and someone accepted that PR
You have to know that you must read line by line until read the title, and then make something like x === "" or x.startswith('http')
Clarifying this point, could someone reopen, or whatever, opens the issue

You're completely right about this. It's an edge case and I've had trouble figuring out the best way to validate this line. Unfortunately it's not supported by current yaml. Perhaps the best way forward is just to move it up so that we always have it in a standard place (for now).

I don't understand your assertion:

The YML seems like a Node env, it received issue.data.body, it has a split, and it seems like a string, what's my point, Javascript support iterate an array, when you reach the same line could work, besides, you have known the next title and you could use a break when you see it in a conditional
If you would want to hardcode the line of the URL, it must be the first section, so, nobody could change the line number
Even if I tried to resubmit this issue, it wouldn't work because I'm providing two screenshots, I could re-submit the issue by leaving the image in another section, but:

It's inaccurate
It enforces ignore issues and nobody is checking them
You could think this is the responsibility of whoever opened the issue, actually, this is a paid service, and Google must provide good services to their customers, I had a least 20% failure and I had to switch to another solution, so, you have to understand the importance of this issue, Google potentially keeps losing a client due to this policy, I thought in remove the support to Google Text to Speech in favor of another, what am I trying to say you? this is an issue with a big importance

Same issue:

https://github.com/googleapis/google-api-nodejs-client/issues/3591
https://github.com/googleapis/google-api-nodejs-client/blob/main/.github/scripts/close-invalid-link.cjs
https://issuetracker.google.com/issues/380295064

Hey, the policy to report issues in Google is so disgusting, it's mentally exhausting, a Google Team must be able to send reports to another Google Team, I mean, I am not a Google worker, I am forced to deal with it
What I reported?, just see

I am frustrated, I want to never send any issues again

Hi @jefer94, as I have already described to you earlier and in #5833, you are right and I need to fix this - I haven't had time to do it this week. I have already marked this issue as a p1 and acknowledged it. However, I will be closing this one and opening another one in its stead because you are violating our code of conduct. You are right there is an issue and I will fix it - please remember there are humans behind this issue queue and we are doing our best :)

@sofisl I am not saying that I'm frustrated about you. I'm frustrated about Google Policy, I wrote about that issue because it is related to this issue
Addressing that I am "violating our code of conduct," I will split the mistake by each one because it seems like you are telling me that I did something related to hating someone or laughing about newbies
About @sofisl

the only thing that you must do is work the stipulated work hours, I’m not saying you have to work twice to resolve our issues

About Google

Google must keep the quality of its products, and respect its customers and beta testers
Google is responsible to be fear of they customer and the beta testers
Google is responsible for contract more developers if needed
Google is responsible for its policy and bureaucracy

About me

As you @sofisl I'm working, if someone doesn't work I must leave a constancy of it
Maybe if I explained what happened wrong, but the error was not only a logic error, it's too don't have empathy, then I said that could be acceptable I didn't put so much importance in it, but and someone accepted that PR I'm making a strong point in that quality assurance fails

Right now clarifying those points, @sofisl , do you think that is fear everything I had to go through? Nothing starts or ends in you, who is in charge of maintaining and enforcing the respect towards the client and beta tester? We are human beings too and we have been going through a nightmare because Google has been systematically disrespecting us
I will leave all that happened to me, for everyone to judge if this is fair and maybe add a clause to our code of conduct about Google respecting our, because we are human beings
https://github.com/googleapis/google-cloud-node/issues/5822#issuecomment-2484494317
https://github.com/googleapis/google-cloud-node/issues/5823#issuecomment-2484505344
https://github.com/googleapis/google-cloud-node/issues/5824#event-15342470199
https://github.com/googleapis/google-cloud-node/issues/5833#event-15377330425
https://issuetracker.google.com/issues/378688996
https://issuetracker.google.com/issues/380282494
https://issuetracker.google.com/issues/380295064
https://github.com/googleapis/google-cloud-python/issues/13194#event-14829117978
https://github.com/googleapis/google-cloud-python/issues/13102#issuecomment-2477153322
https://github.com/googleapis/google-api-nodejs-client/issues/3591#event-15401773702
I suppose that it's fair


🐛 Bug Report: problem with graphQL queries in Dart

👟 Reproduction steps
final tryGraphQL = await ApiClient.graphQL.query(query: {'query':
    """

  query {
    databasesListDocuments(
        databaseId: "$appwriteDatabaseId",
        collectionId: "$collectionId",
         queries: ["equal("name", "myname")"]
             ) {
                 total
                 documents {
                  _id
                  data
                  }
                }
}
  """
    });

it's work if I remove queries: ["equal("name", "myname")"]
👍 Expected behavior
get data from database
👎 Actual Behavior
{errors: [{message: Field "databasesListDocuments" argument "queries" requires type String, found name., extensions: {category: graphql}, locations: [{line: 6, column: 51}]}, {message: Field "databasesListDocuments" argument "queries" requires type String, found myname., extensions: {category: graphql}, locations: [{line: 6, column: 59}]}]}
🎲 Appwrite version
Different version (specify in environment)
💻 Operating system
Linux
🧱 Your Environment
appwrite: ^12.0.4
appwrite server: self-hosting 1.5.7
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

@TechComet, thanks for raising this issue! 🙏🏼 Where did you see that syntax for the queries? That's the old syntax which would result in an error. The new queries syntax can be found here. Also, the nested double quotes you have would probably lead to problems too.


Other private npm solutions

Hi,
First, thanks for your active work on maintaining sinopia!
We are currently looking in my company at possible solutions for private npm/npm cache and of course verdaccio/sinopia is being considered. At the same time we found some alternatives that seems to be actively used by others:

Nexus 3: https://books.sonatype.com/nexus-book/3.0/reference/npm.html
CNPM: https://github.com/cnpm/cnpmjs.org
Artifactory: https://www.jfrog.com/confluence/display/RTF/Npm+Registry

Is anyone using/considering one of those?

I've tried artifactory, but it doesn't handle being used as a staging registry for packages that eventually get published to npmjs.org. The problem comes up when you publish<EMAIL_ADDRESS>to it and then later publish<EMAIL_ADDRESS>to npmjs.org. When you go to install from it later it uses the metadata from npmjs.org, including the shasum, but the actual tarball is fetched from it's internal storage. Unless you jump through some hoops and bypass the normal npm publish flow, the tarballs will not be identical so you end up getting a permanent shasum mismatch.

Closing, is not a bug, but keep the discussion, good to know about more alternatives

https://github.com/craftship/codebox-npm (Used to be called Yith)

@matiaslarsson done 👍


WEBSITE BUILT BY TOBIAS FRIED.

@bqdong thx!


Update integers.ipynb

change "-" to "*" in line 119

change "-" to "*" in line 119

Thanks! Make sure src/notes/Integers.jl is also fixed.


Java libraries do not work in Android due to GRPC version compatibility issues

I need to call AutoML tables via an Android app for live predictions. I tried using the client library, but when I sync the build.gradle in Android Studio, I get the following error:
ERROR: In project 'app' a resolved Google Play services library dependency depends on another at
an exact version (e.g. "[1.21.0]", but isn't being resolved to that version. Behavior exhibited by the
library will be unknown.
Dependency failing: io.grpc:grpc-okhttp:1.21.0 -><EMAIL_ADDRESS>but grpc-core
version was 1.25.0.

I do not know if I can change the GRPC version or that if it is a good idea. A probable solution is to use the REST API, but it gets cumbersome due to expiring access tokens. Is there a way I can get around this ?

Import libraries-bom: https://github.com/GoogleCloudPlatform/cloud-opensource-java/wiki/The-Google-Cloud-Platform-Libraries-BOM


icon-data support for dbusmenu

You didn't dispose the MemoryStream

@Kermalis disposing MemoryStream is a no-op.


Garbled

If I use chinese post title will show ?????????
maybe website need to set utf8 by default
GeniXCMS 1.0.2

it's utf8 by default


change to documentation: RHO_UN function

Reported by ansley b manke on 19 Apr 2010 18:00 UTC
James Orr writes:
Dear Ansley,

I write you in regards to incomplete documentation in the
Ferret Users Manuel about the RHO_UN function and trying to
 improve it.  I am also copying Billy Kessler, because he
has responded previously to the Ferret list in regards to
questions about RHO_UN and I hope he can verify my concerns
 and proposed solution.

Recently, I had a question from a student here, a new ferret
user, wondering why that function gave some unusual results.

His problem was that for the 3rd argument in

   RHO_UN( salt, temp, P )

he was using in-situ pressure instead of the reference
pressure.  Yet, I know from experience and interaction with
 other ferret users (including Billy) that the 3rd argument
 must be the reference pressure. Unfortunately, in the
Ferret user's manual, the third argument is described only
as being "pressure".  Might it be possible to change part of
the first sentence in the RHO_UN documentation from "and
pressure P(dbar)" to "and reference pressure P(dbar)" ?

Additionally, a simple example should make it very clear to
all. Furthermore, if the example could be preceeded by both
(1) a conversion from the IT90 to the ITS68 temperature
scale and (2) a call to THETA_FO (to get potential

temperature before calling RHO_UN), there would no more room
for conceptual misunderstandings. As a starting point, I
would suggest the following example:

! Convert from IT90 to ITS68 temperature scale
  t68 = 1.00024 *  temp
! Get potential temperature
  let potemp0 = THETA_FO(salt, t68, P, 0.)
  let potemp2 = THETA_FO(salt, t68, P, 2000.)
! For sigma0:
  let rho0 = rho_un(salt, potemp0, 0.)
  let sigma0 = rho0 - 1000.
! For sigma2:
  let rho2 = rho_un(salt, potemp2, 2000.)
  let sigma2 = rho2 - 1000.

Actually, in the 2nd to last ligne above, I'm not sure if we
should be using "potemp2" or "potemp0".  Billy, could you
please comment?

Many thanks!

Jim

And Billy replies:

Hi Jim -

You are absolutely correct that the temperature in a call to
RHO_UN should be potential temperature found using THETA_FO,
as in your definition of sigma0 (which I would call sigma-
theta). For general oceanographic use we would always use
sigma-theta (or sigma referenced to a particular pressure
level).

But frankly, I am unequipped to advise about your definition
of sigma2! My work is generally in the upper 1km, so I
always use sigma-theta. I am almost certain your definition
(using potemp2) is correct, but I don't want to mislead you
in case there is a subtlety I am unaware of. This should be
discussed in most oceanographic textbooks, but as it happens
I am also in France now and don't have any textbooks at hand
to check.

Regards .... Billy K

So, it seems correct to change the description of the function in the documentation (and also I suppose in the function description that Ferret gives...)
    yes? show func rho_un
    RHO_UN(salt,temp,p)
        UNESCO state equation (density) for ocean H2O (kg/m^3)
        salt: salinity (psu)
        temp: temperature (deg. C)
        p: pressure (decibars)

where the last line would now say "reference pressure".
Will consult further with James and Billy about putting a new example in the Users Guide.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1734

Comment by ansley.b.manke on 20 Apr 2010 20:09 UTC
Further email from James:
Since yesterday, I've checked around and compared the sigma2
calculation in Ferret (the example I sent you previously)
against the widely used CSIRO Matlab Routines for the
properties of seawater, available at

  http://www.cmar.csiro.au/datacentre/ext_docs/seawater.htm

The numerical results with my example in Ferret are entirely
consistent with those from the Matlab routines. Furthermore,
I've looked into the actual Matlab code (routine pden) and
it takes exactly the same approach. Therefore, it is indeed
correct to use "potemp2" instead of "potemp0" when computing
sigma2.

Ansley, would it be possible to include the following
example in the Ferret Users Guide in the "RHO-UN" section?
Relative to the code sent earlier, this example also defines
3 input variables and adds definitions for sigma-t and
in-situ density (sigma).

! Define input variables:
  let temp=2 ; let salt=35 ; let p=5000
! Convert from IT90 to ITS68 temperature scale
  let t68 = 1.00024 *  temp
! Get potential temperature
  let potemp0 = THETA_FO(salt, t68, P, 0.)
  let potemp2 = THETA_FO(salt, t68, P, 2000.)
! For sigma0 (sigma-theta, i.e., potential density)
  let rho0 = rho_un(salt, potemp0, 0.)
  let sigma0 = rho0 - 1000.
! For sigma2
  let rho2 = rho_un(salt, potemp2, 2000.)
  let sigma2 = rho2 - 1000.
! For sigma-t (outdated method, sigma-theta is preferred)
  let rhot = rho_un(salt, t68, 0.)
  let sigmat = rhot - 1000.
! For sigma (in-situ density)
  let rho = rho_un(salt, t68, p)
  let sigma = rho - 1000.

Billy, any comments would be much appreciated.

Comment by ansley.b.manke on 23 Jul 2010 23:39 UTC
pressure is changed to reference pressure both in the Users Guide and in the function description listed on "SHOW FUNCTION RHO_UN".


All Components | Standardize data attrib -hover (no more mouseenter or mouseover)

The interface presented by the data attribs should universally use HOVER regardless of the actual implementation in the code. It should hide the complexity. The Wired4Sound component is one example of a bad / inconsistent data attrib implementation.
THIS IS A BREAKING CHANGE: Will need to change index.html, wiki, landing page, and all alpha sites.

This is fixed in v3.0. THIS IS BREAKING. The landing page and TSK has been updated.


Add spritefonts support

The import font dialog is good and it works well with existing fonts. But when I have to draw my own font I actually find it hard to import it.
The BMFont app from Angelcode to generate .fnt fonts doesn't work and one needs to script a little to create a custom font.

What's the status on this issue? Is there something we can do, or can someone confirm that there's no standard format to work with?

As a note Godot supports importing of BMFont format. There's no other standard AFAIK, but there's also the possibility to make custom import plugins (I even made one for old XNA SpriteFonts).

@akien-mga lease reopen this issue. I would like to not have to use a 3rd party tool to make sprite fonts when I can just import a sprite texture and define the string map characters and v/h spacing.


Incorrect indent of expression arrow functions with multi-line parameter list

TypeScript Version:  2.4.1 / nightly (2.5.0-dev.201xxxxx)
Code
const exprFunc = (
) => (
  undefined
);

const stmtFunc = (
) => {
  undefined;
};

Expected behavior:
const exprFunc = (
) => (
  undefined
);

Actual behavior:
const exprFunc = (
) => (
    undefined
  );

Works as requested now


Can we gitignore db/structure.sql on ohm-website or are changes meaningful

When we're doing local dev on ohm-website, there are often changes showing up on https://github.com/OpenHistoricalMap/ohm-website/blob/master/db/structure.sql for reasons that are not clearly tied to any particular changes we are making on the site.
We have so far just been excluding that file from any commits, but here's an example where I accidentally committed it to a simple PR about something else entirely:
https://github.com/OpenHistoricalMap/ohm-website/pull/155/commits/3dbcffb5c86be9d7be12552e55017786a366d537#diff-1dd5a8f580b9615769ddc19a64a98f455d9d20eb149c443f41d9d3c797d39b12
So:

What is the function of this file and what actions change it? Should we be doing anything differently to avoid changing it?
Should changes be committed?
Or can it be git-ignored without breaking anything?

I wouldn't add it to gitignore because we want to pull from upstream when needed. But during local dev it can be ignore while pushing changes. The changes are most often mean that the migrations weren't completed on the local db.

I see. Huh. So what's the impact of accidental commit of this file? I guess that's what we need to document:

should local dev work always, sometimes, or never commit changes to this file?
if sometimes, how do we determine if the changes are significant?

Going to leave this as "can't be ignored, shouldn't be committed, unknown how to determine if it should ever be committed." i imagine commitable changes to this file would come only from DevSeed or upstream.


Add verbose configuration to conversational replies

this is annoying

this will require changing the signature of FollowUpConversation to a tuple (in galaxygpt-api). item1 being the new conversation, item2 being the context.
then, CompleteChatPayload needs a new field for the context. but even then, is this viable? we still dont really know what to do with the context.. i'm honestly thinking we should put this on the backburner until we figure out contexts
theres just too many questions that get raised when implementing this in the current state of the repo


Remove .button class usage from surveys and use components

What
Removing .button class usage on survey pages and updating the forms to use components.
Why
We're trying to remove this class from static (pulled in from toolkit) and components should be used to make forms like these ones more accessible.
Changes
See individual commits for specific changes.
Pages changed:

/done/register-flood-risk-exemption (digital survey - Send feedback button)
/done/waste-carrier-or-broker-registration (digital survey - Send feedback button)
/done/register-waste-exemption (digital survey - Send feedback button)
/done/check-settled-status (standard survey - Send feedback button)
/done/transaction-finished (Join button)
/done/driving-transaction-finished (Join button)

Visual changes (assisted digital satisfaction survey)
There's some form elements that get shown and hidden depending on what options are chosen, which follow after each other in the original page. Using components these options are now nested using the conditional reveal functionality of the radio component.
Note that there's an input element for 'other' which appears unstyled. This is because static is overriding the styles from the component. Since this isn't utterly horrible and we're currently working to remove these styles, I've decided to leave this for now rather than add an override.

Before
After

Visual changes (standard survey)
There's no show/hide functionality on this page.

Before
After

@vanitabarrett have had another look at this and expanded the PR somewhat. Please can I get a review when you have time, thanks.


ECAM  550.65w but with problems

Hi, thanks for integration.  Works with ECAM  550.65w  but with some issues

BT connection isn't stable
if you have a personal profile with custom drinks, for example, when you pulse  button prepare Espresso, is the espresso standar, but no custom expresso.

Sorry for my english
Regards

Hey, if you have a problem with bluetooth use extra 1m cable. It hale me to fix issue with swapping bluetooth connection.

Ok, bt connection solve, but not custom drink. Also, I can't do drinks with milk

please check this PR https://github.com/Arbuzov/home_assistant_delonghi_primadonna/pull/18 maybe it will help you. But now I see that this plugin not support all functionality - which I do not judge because I understand that it is not easy to write such a plugin 😀

The integration should have profile support now.


encoders and hashing

Problem
In  https://github.com/golang-migrate/migrate/blob/master/README.md contains this:
It's easiest to always run the URL parts of your DB connection URL (e.g. username, password, etc) through an URL encoder. See the example Python snippets below:

$ python3 -c 'import urllib.parse; print(urllib.parse.quote(input("String to encode: "), ""))'
String to encode: FAKEpassword!#$%&'()*+,/:;=?@[]
FAKEpassword%21%23%24%25%26%27%28%29%2A%2B%2C%2F%3A%3B%3D%3F%40%5B%5D
$ python2 -c 'import urllib; print urllib.quote(raw_input("String to encode: "), "")'
String to encode: FAKEpassword!#$%&'()*+,/:;=?@[]
FAKEpassword%21%23%24%25%26%27%28%29%2A%2B%2C%2F%3A%3B%3D%3F%40%5B%5D
$

this is suboptimal for several reasons (e.g. requires python, verbosity, hard to read).
Solution
Provide commands for encode/decode (URL, base64, etc) and hashing (md5, sha1, sha2) by using
Java classes e.g. Base64 or MessageDigest.
Benefit
Easy to use and portable commands for encoding, decoding and hashing.

base64 [ ]
md5 [ ]
sha1 [ ]
sha2 [ ]


UnknownOperation string representation should be UnknownOperationException

https://github.com/awslabs/smithy-rs/blob/c6d6c2b131259ecdbd9bd923a0a04db8c610854f/rust-runtime/aws-smithy-http-server/src/runtime_error.rs#L45-L53
The string representation is UnknownOperationException. See the TypeScript implementation.
I guess we didn't catch this because there aren't any protocol tests. But even if they existed, we wouldn't be able to pass them, since this error arises when the Router is not able to route to an operation, a code path we do not exercise because of https://github.com/awslabs/smithy-rs/issues/1212.
So once we do https://github.com/awslabs/smithy-rs/issues/1212, we should add some ambiguous routing protocol tests.

Fixed the string in https://github.com/awslabs/smithy-rs/pull/1424, keeping this open to add tests

I think we can close this right?

No, we're blocked on #1212 to add protocol tests to not regress on this.

So once we do https://github.com/awslabs/smithy-rs/issues/1212, we should add some ambiguous routing protocol tests.


Restore extension page after reload

When I install a new extension and reload the window the extension page is lost. I have to go to extensions and find it again. I often install extensions and look at it's README only after it's loaded.
Would be nice if the extension page(s) restore after reload.
Also restore the Extensions sidebar with the last query which gets lost as well.

@bpasero How to achieve this?

@joaomoreno that is possible by registering an editor input factory that can serialise an editor input to JSON and then from there back again. One example of such a factory is the one for files: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/files/browser/files.contribution.ts#L113

How do you serialize IExtension (needed to create ExtensionsInput I believe)?

@bpasero @Sakerdot Asks a valid question. Since IExtension has functions, is serializing it the right way? One could serialize the resource URI and then retrieve the data, but that requires reworking other things. Since the "recommended extensions" already has the data. What do you reckon is the best approach to solve this?

@joaomoreno Any thoughts?

Can we just save the id of the extension which has been installed and when the user will press reload button it will set a task to open up the extension page and when vs code will reload it will first check for that task and if its there, it will simply open the page? And if user press reloads after installing multiple extensions then it will open up the page from which user pressed reload button or it can open all pages of all those extensions which were installed in the last session.
After reloading vs code automatically open up the sidebar with extensions tab open maybe it is storing state somewhere and we can use the same.
I am a beginner and wants to work on the feature if that's OK?

Hello @joaomoreno
I am looking to start contributing and would like to start off with this issue.
Is this issue still available?

cc @sandy081

@punteek This is still open, so you can contribute 👍

Hi guys, Looking for a first opportunity to contribute.
Can I contribute to this?
Any pointers to start will help big time. Thanks in advance.

@bhuvinth

You have to register an editor input factory so that the input can be retrieved after reload. You have to store extension ID while serialising since you cannot store complete extension. See https://github.com/Microsoft/vscode/blob/393b48d4a5de85b6fa1d2ad0a59588d6715e12ea/src/vs/workbench/contrib/extensions/electron-browser/extensions.contribution.ts#L111

You have to refactor ExtensionsInput class to accept extension id from the deserialised input and change reload method to get the extension from IExtensionsWorkbenchService

Zuper.. I just read this. I was sick. I am starting to work on this tonight.

Hi, I saw this is task is a good first issue and was wondering where I should get started to work on this. Thanks!

@sandy081 I thought this would be simple but it turns out it is not. The problem I ran into is that ExtensionsInput takes a IExtension object which cannot be easily serialized and deserialized.
Assigning back to you and unmarking as good first issue. I still think this is worth fixing if possible

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Removed snapUpdate.sh and replaced with inline command

This patch removes the need for snapUpdate.sh. This has the following benefits:
+Simplifies the build process by avoiding setting and moving the file.
+Avoids creating a new shell that simply spawns another shell environment.
+Simplifies the update mechanism by removing the need to track the existence of snapUpdate.sh.
+Keeps the logic for snapUpdate.sh close to updateService.snap.ts
I have tested the code changes to spawn within a Node environment in vscode console and it correctly spawns another instance of vscode.
If this patch isn't wanted, there are still some changes that may be suggested. For example, currently updateService.linux.ts is calling snapUpdate.sh but the default Linux builds aren't being built with the script. Similarly, the comment for executing snapUpdate.sh isn't correct.
Running the command 'grep -nr --exclude-dir=".git" 'snapUpdate.sh' .' produces no results so I assume I have removed all references.

Hello @joaomoreno and @Tyriar ,
Would it be possible for you to take a look at my branch Kedstar99/UpdateServiceBranch?
I think it fixes something somewhat serious with this code.
I am happy to discuss further if you want on the commit thread.
Kind regards,
Krish

What does it fix?

https://portal.msrc.microsoft.com/en-US/security-guidance/advisory/CVE-2019-0728
According to this CVE, if an attacker is able to define environment variables, they may be able to cause execution of their code by getting users to just open the project. This was deemed serious enough to get a CVE. I guess the issue in this case, is if this is specific to workspace environment variables and if these get extended to the parent process.
If an attacker is able to set the environment variables for SNAP_REVISION, SNAP and SNAP_NAME, they may be able to get execution through doQuitAndInstall.
In my new patch I removed the use of the environment variable and also removed the snap related code from the LinuxUpdate file. If the user has any of the snap variables set, they would be directed to use the SnapUpdate service instead. As such it's either dead code or a vulnerability (assuming a user would be able to change the environment variables for another running process).
Arguably, if an attacker has control of the environment variables, they have in theory already broken the system because they could just change the path variables to point to their own binary location and get execution whenever vscode calls lsof or cd...
In any case, with the code in my patch, I can be more confident that the above issue with the SNAP variables doesn't occur.

I replaced it with a call to explicitly the current running binary rather than whatever is set as SNAP_NAME.

Isn't that the executable of the older snap version?

By calling path.basename(process.argv[0]), I should get the same effect as SNAP_NAME.

Shall I make a PR for this I guess?

By calling path.basename(process.argv[0]), I should get the same effect as SNAP_NAME. It just returns the name of the current running binary, either code-insiders, vscode etc...

Oh right!

Shall I make a PR for this I guess?

Yeah please go ahead.


File Picker redirect_uri restriction is a big pain point

The file picker SDK is basically missing the point of integration with any serious web application because

the developer can't test the file picker SDK on a local environment, see #11
OneDrive requires the source URL to be static. All dynamic source URLs, e.g. /collections/{id}, can not be used as it is not possible to pre-register every "flavour" as redirect url in the* "Microsoft account Developer Center"*.

You can read up on frustrated developers encountering these issues on stackoverflow from 2014 and across this repository.
These limitations render the file picker SDK useless for any use case.
I also tried the new v6.0 SDK but that seem to conflict with my client id: AADSTS70001: Application '3645e8ff-35d6-xxxxxxxxxxxxxxxxxxxx' is not supported for this API version.. Obviously, there are no resources on why this is or how this can be resolved. And I'm not the only one:

check this
this
or this unresolved question
or do a quick search on Google

I am sorry if this post feels like a rant, but this is one of the most frustrating developer experiences I had during the last month and the most frustrating in comparison to all other leading cloud drive providers, including dropbox and google drive amongst others.
If I am mistaken, please point me at the resources that resolve this. One week of digging through developer resources I am still left confused with all the different Web SDKs / APIs and missing well explained steps to implement the most basic things.

Hi @arekkas!
Thanks for the genuine feedback. We're certainly trying to make the developer experience easier, but we still have some places where things are broken. The "app is not supported for this API version" appears to be an error coming from the Azure Active Directory API, so I'll pass this feedback along to the team that owns it. However, I wouldn't expect this issue to be encountered when using our JS Picker SDK since it only calls the OAuth end point for AAD. If you can get a trace (Fiddler is our favorite tool) of a scenario where you get this error message we can debug what's going on a lot easier.
In the time since we've answered and closed #11, the team that owns identity has actually enabled localhost URLs to be used as redirect URLs for registered apps. If you've registered an app in Azure Active Directory, you can use register redirect URLs for both localhost and your domain in the same application. For Microsoft Account registered apps, you can only register one or the other (so you'll need to use separate app IDs for localhost testing and for production, which is a good idea anyway).
The feedback about the redirect URL for the file picker not support dynamic URLs is something we're addressing for the next update. We'll allow the developer to specify a custom redirect URL that doesn't have to be the page hosting the file picker to make sure that it can be supported on more dynamic pages than the current SDK allows.
We're also working to incorporate Microsoft Graph API and App Model v2.0 into the next version of the JS file picker SDK to streamline the authentication story and only require a single app registration (instead of needing two client_ids to support AAD + MSA). We're working to get this next version out as a refresh to the current preview over the next couple of weeks.
Again, thank you for sharing your voice and helping us find the places that we can do better.

I @rgregg thank you for responding to this issue so timely and honest. I will take a closer look at Azure AD as I'm currently using a Microsoft Account app and I will try to figure out why localhost fails as redirect URL on my end - I guess it has to do with declaring a variety of ports (3000, 4567, ...) on localhost as redirect endpoints.
I am very glad that these kind of issues are taken seriously and am happy to be able to contribute in making the developer experience easier. Please let me know if I can help other than pointing at issues I stumble upon.
I will deep-dive into Azure AD and the other things you mentioned and come back with hopefully positive news once I am able to get everything working as it should.

I didn't have the time to go through AD yet and I'm not confused if AD costs money for the use case described above.
I was however able to get the redirect stuff working by registering one app per port. Thanks for pointing this out!

AAD is free, although you need either a free trial subscription for Azure or a pay-as-you-go subscription. As long as you only use the Azure AD feature, you wouldn't be billed, but I'm glad you were able to get things registered using different ports.

Perfect, thanks!

@rgregg Is there any ETA on the dynamic URL support of redirect URI's?

redirect URL for the file picker not support dynamic URLs still not working. Even option:
advanced: {
redirectUri: "https://contoso.com/filePickerRedirect.htm"
}
does not work at all. Any update?


web-proxy: Enable Linux Package Cache is not documented

The new feature "Enable Linux Package Cache" has a too brief description that looks more likea marketing message than a technical doc:

Enable or disable the caching of packages for linux distributions. This makes sense if you have multiple servers in your network and do not host your own package mirror. This will reduce internet traffic usage but increase disk access.

This does tell the user how the caching would function. Also the official product documentation does not include any mention of this feature.
Please provide more information regardin the outcome of this option as in which condition it would work or not.

https://github.com/opnsense/core/blob/47a3b2419d4dfd994b2c7d79b7cbacc7f457d4e8/src/opnsense/service/templates/OPNsense/Proxy/squid.conf#L319-L329


GUI for this

I was working on a GUI for music player and wanted to obtain tags for music and came across this, so I found out GUI is inside your todo list, so maybe I could help, with tkinter GUI. Well to know more about the GUI I make, just visit my github repo and also anyway to pinpoint tracks? Using the search_track it finds inaccurate tracks. Any ideas to point on exact song on searches like 'Justin Bieber & benny blanco - Lonely'. Thanks :D

Hi, thanks for taking interest on contributing to this repo.

Using the search_track it finds inaccurate tracks.

I see when you search using that query: Justin Bieber & benny blanco - Lonely, it gives the piano version as the first result. That is intended, if you search the same query on the official Deezer it will give similar results. The way I see it, to have an accurate result, you should input "less" specific keywords. If I want to search Justin Bieber's Lonely featuring Benny Blanco, I'll search for lonely - justin bieber.

Any ideas to point on exact song on searches

If you really want to have an EXACT match. The only idea I can think of is to get the track's id from Deezer and get the track using the get_track(track_id) method.

Hi, thanks for taking interest on contributing to this repo.

Using the search_track it finds inaccurate tracks.

I see when you search using that query: Justin Bieber & benny blanco - Lonely, it gives the piano version as the first result. That is intended, if you search the same query on the official Deezer it will give similar results. The way I see it, to have an accurate result, you should input "less" specific keywords. If I want to search Justin Bieber's Lonely featuring Benny Blanco, I'll search for lonely - justin bieber.

Any ideas to point on exact song on searches

If you really want to have an EXACT match. The only idea I can think of is to get the track's id from Deezer and get the track using the get_track(track_id) method.

Is there a way to get track id, from deezers site? or something?

Is there a way to get track id, from deezers site? or something?

Yes, definitely. All the tracks and search results shown in Deezer is an anchor tag and you can take its id in its href.
Lonely - Justin Bieber, Benny Blanco have this link: https://www.deezer.com/en/track/1110767312. The last part is actually the track's id so<PHONE_NUMBER>.

Yes, definitely. All the tracks and search results shown in Deezer is an anchor tag and you can take its id in its href.
Lonely - Justin Bieber, Benny Blanco have this link: https://www.deezer.com/en/track/1110767312. The last part is actually the track's id so<PHONE_NUMBER>.

Sorry for continuing this issue, but how do you get the link, the website seems to have a share link for the song and that is https://deezer.page.link/xrYh93mFnT3xYvZt7.

Sorry for continuing this issue, but how do you get the link, the website seems to have a share link for the song and that is https://deezer.page.link/xrYh93mFnT3xYvZt7.

Im sorry for continuing this issue, but how do you get the link, the link that can be shared from deezer is https://deezer.page.link/xrYh93mFnT3xYvZt7.

Im sorry for continuing this issue, but how do you get the link, the link that can be shared from deezer is https://deezer.page.link/xrYh93mFnT3xYvZt7.


runtime: On Windows, lastcontinuehandler intercepts exception raised in non-go code

What version of Go are you using (go version)?

$ go version
go version go1.12 windows/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\simon\AppData\Local\go-build
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=c:\gohome
set GOPROXY=
set GORACE=
set GOROOT=c:\go
set GOTMPDIR=
set GOTOOLDIR=c:\go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\simon\AppData\Local\Temp\go-build415554718=/tmp/go-build -gno-record-gcc-switches

What did you do?
I have built a GO program that calls C# code using CGO. When the .Net debugger is attached to the process, when any .net code is called, I get a panic like with the following trace:
Exception 0x80000003 0x0 0x0 0x7ffcd17d4052
PC=0x7ffcd17d4052

runtime: unknown pc 0x7ffcd17d4052
stack: frame={sp:0x87fd58, fp:0x0} stack=[0x0,0x87fdc0)
000000000087fc58:<PHONE_NUMBER>000190<PHONE_NUMBER>030000
000000000087fc68:<PHONE_NUMBER>000000<PHONE_NUMBER>87fc90
000000000087fc78:<PHONE_NUMBER>000003<PHONE_NUMBER>000001
000000000087fc88:  00007ffcd3beae06<PHONE_NUMBER>000004
000000000087fc98:<PHONE_NUMBER>004000  00007ffcd3d4d000
000000000087fca8:<PHONE_NUMBER>000003  00007ffcd3d4d3e0
000000000087fcb8:  00007ffcd3c4ed63<PHONE_NUMBER>980740
000000000087fcc8:  000000c00004fed0  000000c00004fed0
000000000087fcd8:<PHONE_NUMBER>4c0000 <flag.(*FlagSet).parseOne+880><PHONE_NUMBER>000000
000000000087fce8:  000000c00004ffd0<PHONE_NUMBER>000000
000000000087fcf8:  000000c00004ffc0<PHONE_NUMBER>030000
000000000087fd08:<PHONE_NUMBER>000000  000000c000050000
000000000087fd18:<PHONE_NUMBER>5070e5  000000c00004fed0
000000000087fd28:  000000c00004fe90  000000c00004fed0
000000000087fd38:<PHONE_NUMBER>000000  000000c00004fed0
000000000087fd48:<PHONE_NUMBER>6595c0  000000c00004fed0
000000000087fd58: <000000000045b9f3 <runtime.asmcgocall+115><PHONE_NUMBER>000000
000000000087fd68:<PHONE_NUMBER>000000<PHONE_NUMBER>000000
000000000087fd78:<PHONE_NUMBER>000001  000000c00004fe08
000000000087fd88:<PHONE_NUMBER>0001a0  000000c00003f200
000000000087fd98:<PHONE_NUMBER>434bb0 <runtime.mstart+0><PHONE_NUMBER>172480
000000000087fda8:<PHONE_NUMBER>45a02c <runtime.rt0_go+332><PHONE_NUMBER>172480
000000000087fdb8:<PHONE_NUMBER>45a033 <runtime.rt0_go+339>
runtime: unknown pc 0x7ffcd17d4052
stack: frame={sp:0x87fd58, fp:0x0} stack=[0x0,0x87fdc0)
000000000087fc58:<PHONE_NUMBER>000190<PHONE_NUMBER>030000
000000000087fc68:<PHONE_NUMBER>000000<PHONE_NUMBER>87fc90
000000000087fc78:<PHONE_NUMBER>000003<PHONE_NUMBER>000001
000000000087fc88:  00007ffcd3beae06<PHONE_NUMBER>000004
000000000087fc98:<PHONE_NUMBER>004000  00007ffcd3d4d000
000000000087fca8:<PHONE_NUMBER>000003  00007ffcd3d4d3e0
000000000087fcb8:  00007ffcd3c4ed63<PHONE_NUMBER>980740
000000000087fcc8:  000000c00004fed0  000000c00004fed0
000000000087fcd8:<PHONE_NUMBER>4c0000 <flag.(*FlagSet).parseOne+880><PHONE_NUMBER>000000
000000000087fce8:  000000c00004ffd0<PHONE_NUMBER>000000
000000000087fcf8:  000000c00004ffc0<PHONE_NUMBER>030000
000000000087fd08:<PHONE_NUMBER>000000  000000c000050000
000000000087fd18:<PHONE_NUMBER>5070e5  000000c00004fed0
000000000087fd28:  000000c00004fe90  000000c00004fed0
000000000087fd38:<PHONE_NUMBER>000000  000000c00004fed0
000000000087fd48:<PHONE_NUMBER>6595c0  000000c00004fed0
000000000087fd58: <000000000045b9f3 <runtime.asmcgocall+115><PHONE_NUMBER>000000
000000000087fd68:<PHONE_NUMBER>000000<PHONE_NUMBER>000000
000000000087fd78:<PHONE_NUMBER>000001  000000c00004fe08
000000000087fd88:<PHONE_NUMBER>0001a0  000000c00003f200
000000000087fd98:<PHONE_NUMBER>434bb0 <runtime.mstart+0><PHONE_NUMBER>172480
000000000087fda8:<PHONE_NUMBER>45a02c <runtime.rt0_go+332><PHONE_NUMBER>172480
000000000087fdb8:<PHONE_NUMBER>45a033 <runtime.rt0_go+339>

When using .Net core instead of the full .Net framework, the same problem occurs as soon as a breakpoint is hit in C# code.
I can also reproduce the error by attaching Visual C++ debugger and using CGO to call C code calling DebugBreak
What did you expect to see?
Using another language debugger on a GO program to break on code called using CGO should work without crashing the go runtime.
What did you see instead?
Using another language debugger on a GO program (to debug non-go code called using CGO), crashes my program any time an exception of type EXCEPTION_BREAKPOINT is raised.
Candidate fix
This behavior is fixed in https://github.com/golang/go/pull/32574, although it might have undesired side effects I am not aware of.

/cc @ianlancetaylor

Thank you for raising this issue @simonferquel
Unfortunately I still don't see what your problem is. Can you, please, provide all steps I need to reproduce it. Including all source code and tools I need.
Thank you.
Alex

@alexbrainman sure, no problem. I'll setup a github repo with a repro, and a quick video highlighting the issue. Thanks for taking the time to investigate this with me, this could open the door to better cross-language interop and debugging experience.

Here is the repro: https://github.com/simonferquel/golang-continue-handler-repro
I've added a walk-through video in it explaining the issue (sorry for my english in there, I am not a native english speaker).
If you want to run the example, you'll need:

a Windows machine (or VM)
dotnet core SDK https://dotnet.microsoft.com/download
Visual Studio Code
C# VS code extension: https://marketplace.visualstudio.com/items?itemName=ms-vscode.csharp

VS code tasks are setup, so that hitting ctrl+f5 will start without debugging, and f5 will start with the dotnet core debugger (add a break point at https://github.com/simonferquel/golang-continue-handler-repro/blob/master/csharp/Program.cs#L19, to reproduce the issue).
The breakpoint is hit correctly, but resuming execution crashes the program, because the BREAKPOINT exception is incorrectly trapped by the lastcontinuehandler.

Here is the repro: https://github.com/simonferquel/golang-continue-handler-repro
I've added a walk-through video in it explaining the issue

I can see the problem, thank you very much.  This is very different - C# runtime should be in control of what happens inside of C# process, not Go runtime. I will comment on https://go-review.googlesource.com/c/go/+/181839 about what to do next.
Alex

Hmm that makes perfect sense. How can I determine that I am running a dll? Some global var in runtime?

got it: https://github.com/golang/go/blob/8f296f59de0703b0559474beb434a265e277bdca/src/runtime/runtime2.go#L936

If you use x86  , that will be good!


[graphiql] Disable fetch on initial loading

It would be nice to have the possibility to disable fetch on the first/initial loading of the component.
I see no need for this, and I want to disable this behavior
I didn't find this info in the documentation. Maybe I missed something?

@palyvodaBoi so you don't want autocompletion, validation or hover? you can just supply an empty GraphQLSchema to schema prop then

@acao, it seems like this is what I need. Thank you for such a quick response!


UI: Containerset nodes receive the wrong podname

Pre-requisites

[X] I have double-checked my configuration
[X] I have tested with the :latest image tag (i.e. quay.io/argoproj/workflow-controller:latest) and can confirm the issue still exists on :latest. If not, I have explained why, in detail, in my description below.
[X] I have searched existing issues and could not find a match for this bug
[X] I'd like to contribute the fix myself (see contributing guide)

What happened/what did you expect to happen?
When trying to get the logs for a (any) containerset container in the UI, the user is faced with a blank page.

This is due to the podname being incorrectly calculated for these nodes, which makes the request to the backend return an empty response.
Expected behavior would be to be able to view the logs in similar fashion as for pod nodes.
Version
latest
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: graph-
  labels:
    workflows.argoproj.io/test: "true"
  annotations:
    workflows.argoproj.io/description: |
      This workflow demonstrates running a graph of tasks within containers in a single pod.
    workflows.argoproj.io/version: ">= 3.1.0"
spec:
  entrypoint: main
  templates:
    - name: main
      containerSet:
        containers:
          - name: a
            image: argoproj/argosay:v2
          - name: b
            image: argoproj/argosay:v2
            dependencies: ["a"]
          - name: c
            image: argoproj/argosay:v2
            dependencies: ["a"]
          - name: d
            image: argoproj/argosay:v2
            dependencies: ["b", "c"]

Logs from the workflow controller
N/A

Logs from in your workflow's wait container
N/A

Hmm this seems like a duplicate of #10157; @isubasinghe I thought you tested that this was working?

@agilgur5 I did test if the live logs were showing, this sounds like a different issue.

The issue here is that the initial pod name is being incorrectly calculated, which gives the user empty logs when pressing the 'logs' button. If the user then manually selects the pod for their container they do indeed see the logs.

Right that makes sense, thanks. I think I was selecting the containers individually when I tested #10157 so never noticed this issue. Thanks, will have a look into your PR.

To be clear; logs are not currently shown when selecting containers individually. You can show them by manually selecting the pod for the selected container.


Improve efficiency of ETW metrics exporter

Fixes #104
Design Decisions:

Follow .Net and C++ implementations and use helper methods to serialize fragments of metric points into a coherent, exportable buffer
Construct a new ResourceMetrics for each metric point and serialize each into an exportable buffer

Changes
Please provide a brief description of the changes here.
Merge requirement checklist

[ ] CONTRIBUTING guidelines followed
[ ] Unit tests added/updated (if applicable)
[ ] Appropriate CHANGELOG.md files updated for non-trivial, user-facing changes
[ ] Changes in public API reviewed (if applicable)

Comparing the latest exporter benchmark results between the proposed export implementation and the previous export implementation, there is a ~26% improvement.

Comparing the latest exporter benchmark results between the proposed export implementation and the previous export implementation, there is a ~26% improvement.

With a more realistic ResourceMetrics, the improvement looks to be closer to 38%


Add Broadcast->isOnAir and Service->isActive

Add clarifying comments on if start/end times for services and
broadcasts are inclusive or exclusive (starts are inclusive,
ends are exclusive)

How does this deal with the switch between BST and GMT? E.g. if a broadcast starts at 01:45 and ends at 01:15?

Logic looks sound, I hope there aren't as many nasty lurking timezone issues in DateTime as in our old code, there shouldn't be really. I agree isActiveOn makes more sense if a date is being passed in.

All times should be in the UTC timezone, with a TZ signifying it as such. We shall deal with the conversion to UK time (GMT or BST) at render time.
Shall update to isActiveOn. Should I change isOnAir to isOnAirOn for the same reasons? That repetition  of 'on' looks odd, 'isBroadcastingOn`?
Hopefully the move to usingDateTimeImmutable shall help avoid much of the weird around mutable date instances.

Have updated the functions to be isActiveAt and isOnAirAt and am passing in $dateTime to both of them


Query string delimiting character not working as expected

Regarding this portion of the Step 3, the 'Test the function' section:
Add the query string value ?name=<your_name> to the end of this URL and press Enter to run the request.
The function returns a 401 when using the ? character as indicated.  The function runs successfully when using a & character in place of the ?.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6cf91d8c-d16b-346d-759b-f3c787dea2e5
Version Independent ID: 7e422baf-0a19-dded-ee83-a71b1c819f2f
Content: Create your first function in the Azure portal
Content Source: articles/azure-functions/functions-create-first-azure-function.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@CalebPaul, Thanks for the feedback! We are taking a look into this and will get back to you soon.

@CalebPaul, Thanks for the feedback! We are taking a look into this and will get back to you soon.

@CalebPaul Thanks for reporting this item! I have updated the doc and it will be published soon. We appreciate the feedback!

@CalebPaul Thanks for reporting this item! I have updated the doc and it will be published soon. We appreciate the feedback!

@CalebPaul We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@CalebPaul We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Thank you!

Thank you!

@CalebPaul it seems like you created a function using function access keys instead of anonymous access. This is why you already have the ? for the ?code=... parameter that is the key.

@CalebPaul it seems like you created a function using function access keys instead of anonymous access. This is why you already have the ? for the ?code=... parameter that is the key.

@ggailey777 Thank you for the clarification.

@ggailey777 Thank you for the clarification.


level remote 8 does not explain 'reset' command in intro but that command is part of the level's challenge

Issue observed:
The challenge in level 'remote 8' requires the usage of a new command that has not been mentioned anywhere before so it's hard to solve the challenge because we don't know that this command is expected to be used.
Detailed explanation of the observed issue:
Level Remote 8 explains these three things before we can proceed to solving the challenge:

But the challenge's solution includes the usage of the "reset" command (git reset --hard o/main) which is not mentioned in the explanations seen above or in any previous intros. In the screenshot below, we can see that "show solution" includes this command:

Expected behavior:
The git reset --hard command should be part of the initial intro which usually explains all the new commands that need to be used in order to complete the level's challenge. I tried solving the challenge with only the commands that were explained so far but wasn't able to solve the challenge.

@rena-puchbauer The command explanation is described on level 4th of Ramping Up section ("4: Reversing Changes in Git")


assert() statements across the sub-project code have side effects

Disabling assert() statements (such as with -DNDEBUG with seems to happen with our "release" builds) breaks the functionality of the binaries produced because there are assert() statements with essential side effects. A fix to move all essential logic outside the assert()'s is already in progress, just documenting the issue.

@hellermf this is done, right?


[tw] Localize Vertical Scaling for Traditional Chinese

Localize Vertical Scaling for Traditional Chinese
Source: https://glossary.cncf.io/vertical-scaling/

Thanks


in replicator not closed connections cause "Too many open files" error in long time run

In each replicator loop, a new connection is got from self.get_conn(device), but it is not closed.
In long time run there will be "Too many open files" error.
As the updater and auditor,Using DiskFileManager to get connection from the pool is a better method.

Thanks @wangyanbn, we will look into making this changes.


[question] new release?

Hi, thanks for your work on this project! :)
Would you consider cutting a new release with the latest fixes? It would be nice to be able to download it from PyPI.
Thanks again!

Yes please.

done guys 0.5.17 👌


Replacement of collection variables not working in JSON body when host name includes an environment variable

Hey there,
we are experiencing some really weird behaviour with the replacement of collection variables in JSON POST bodies when the called URL does include variables aswell. Here a minimal example extracted from our project:
Collections Script
const moment = require('moment');

// dates
const firstOfNextMonth = moment().add(1, 'M').startOf('month')
bru.setVar("FIRST_NEXT_MONTH", firstOfNextMonth.format("YYYY-MM-DD"));

Example 1 - POST {{MY_HOST}}/api/pricing
{
    "persons": [
        {
            "id": 99,
            "roles": [
                {
                    "roleType": "APPLICANT",
                    "payment": {
                        "paymentPattern": "MONTHLY"
                    }
                }
            ]
        }
    ],
    "productBundle": {
        "insuranceDuration": {
            "from": "{{FIRST_NEXT_MONTH}}"
        }
    }
}

Result
This one results in the exact same request body. The variable does not get replaced with the actual value.
Example 2 - POST {{MY_HOST}}/api/pricing
If you cant't see the difference, I just swapped the properties payment and roleType in the JSON body of person[0].roles[0] 🤷♂
{
    "persons": [
        {
            "id": 99,
            "roles": [
                {
                    "payment": {
                        "paymentPattern": "MONTHLY"
                    },
                    "roleType": "APPLICANT"
                }
            ]
        }
    ],
    "productBundle": {
        "insuranceDuration": {
            "from": "{{FIRST_NEXT_MONTH}}"
        }
    }
}

Result
The variable gets replaced with the appropriate value of the collection variable FIRST_NEXT_MONTH.
For Reproduction
I attached my bruno collection, unfortunately the collection scripts don't get exported, therefore you have to add the first code block manually. 💾 Download

Hello,
I have the same problem.
Maybe it depends on whether the pre-request script sets the specified variable, if set var by the previous request its works

Hey @rschwedasi, I couldn't reproduce the bug in the latest version of Bruno v1.20.2. I tried adding the script at both the collection level and the request level, and the request body values were interpolated as expected! Could you please verify this by trying out the latest version of Bruno?

Bugs.json
Hey @sanjai0py , You can try attached collection

Closing this issue as not reproducible.
https://github.com/user-attachments/assets/45949446-3560-4976-ac80-6c8e84fd041e
And @BlechaJosef, in the collection JSON you provided, the pre-request script is currently this: bru.setVar("currentTime",moment()); which should be bru.setVar("currentTime",moment().toString()); We can't assign objects to collection vars.

@sanjai0py
Thank you for the explanation.
I was confused that the console was returning the correct values console.log("pre collection", bru.getVar("requestId"), bru.getVar("currentTime"))


Make label compatible with existing ones

Our current labels start with number for priority and also have a color scheme according to priority: https://github.com/NixOS/nixpkgs/pull/92249
Marvin should also follow the same scheme.

I'd wait for GA version of marvin, since "looking official" implies stronger stability guarantees.

I don't think it should follow that scheme. There is not much to be gained, and the priority number is a major inconvenience when handling those labels. That is especially true for labels that we want people to actively search for. They would now have to remember/look up the somewhat arbitrary number prefix and probably quote the label in the search. Additionally its nice to have a 1:1 match between the labels and the status names.
(Also, not all current nixpkgs label match that scheme.)
Colors could be added of course, I don't mind that.


feat(iam-assumable-role-with-oidc): Add assume role policies

Description
Enable users to set source_policy_documents on the assume role policy.
Motivation and Context
Fixes #361.
As I'm creating IAM OIDC roles, I need to be able to test them by assuming them. Using source_policies is the most generic implem, so other use cases might also benefit this.
Breaking Changes
None.
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[ ] I have executed pre-commit run -a on my pull request

:'(

@awoimbee The code looks good. Please rebase it and let me know. I will merge it.

Thanks @antonbabenko , done

Please fix the failing CI checks, too.