Cannot retrieve ANY data from [hit._source]

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?

[ ] elastic.v2 (for Elasticsearch 1.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[x] elastic.v5 (for Elasticsearch 5.x)
[x] elastic.v6 (for Elasticsearch 6.x)

My source
// elk.go
package main

import (
	"context"
	"log"
	"os"
	"fmt"
	"gopkg.in/olivere/elastic.v6"
	"encoding/json"
	"time"
)

type Flow struct {
	sas  int32    `json:"srs.as"`
	das int32     `json:"dst.as"`
	time time.Time `json:"@timestamp"`
}

func main(){
	errorlog := log.New(os.Stdout, "APP ", log.LstdFlags)

	client, err := elastic.NewClient(elastic.SetErrorLog(errorlog),
		elastic.SetURL("http://<IP_ADDRESS>:9200"),
		elastic.SetBasicAuth("elastic", "xxx"))
	if err != nil { panic(err) }

	searchResult, err := client.Search().
		Index("netflow-*"). // search in index "twitter"
		Sort("@timestamp", true). // sort by "user" field, ascending
		From(0).Size(10). // take documents 0-9
		Pretty(true). // pretty print request and response JSON
		Do(context.Background()) // execute

	if err != nil { panic(err)	}

	fmt.Printf("Found a total of %d hits\n", searchResult.TotalHits())
	for _, hit := range searchResult.Hits.Hits {
		var t Flow
		err := json.Unmarshal(*hit.Source, &t)
		if err != nil {
			panic(err)
		}
		fmt.Printf("Flow id#%s AS PATH %d => %d\n", hit.Id, t.sas , t.das)
	}
}

Output
Found a total of 85872550 hits
Flow id#sG6vBGQBQNJao7vxQIcC AS PATH 0 => 0
Flow id#1JfzBGQBQNJao7vxLWYH AS PATH 0 => 0
Flow id#XozgBGQBQNJao7vxnUuj AS PATH 0 => 0
Flow id#UDZUBGQBQNJao7vxJCGc AS PATH 0 => 0
Flow id#DTZUBGQBQNJao7vxIyFK AS PATH 0 => 0
Flow id#DDpaBGQBQNJao7vxnoui AS PATH 0 => 0
Flow id#QTpaBGQBQNJao7vxvJcG AS PATH 0 => 0
Flow id#qYPQBGQBQNJao7vxLC5B AS PATH 0 => 0
Flow id#X4PQBGQBQNJao7vxPjnr AS PATH 0 => 0
Flow id#c4PQBGQBQNJao7vxLi_g AS PATH 0 => 0

Values t.sas and t.das must be > 0:
cUrl/Kibana Dev tool`s query:
curl -Xget "http://<IP_ADDRESS>:9200/netflow-*/_search?pretty&size=1" -H 'Content-Type: application/json' -d'{ "sort" : [
        { "@timestamp" : {"order" : "asc"}}
    ]
}'

Result:

{
  "took": 634,
  "timed_out": false,
  "_shards": {
    "total": 20,
    "successful": 20,
    "skipped": 0,
    "failed": 0
  },
  "hits": {
    "total": 85876984,
    "max_score": null,
    "hits": [
      {
        "_index": "netflow-0008.06",
        "_type": "doc",
        "_id": "sG6vBGQBQNJao7vxQIcC",
        "_score": null,
        "_source": {
          "dst.as": "11111",
          "packets": "1",
          "type": "netflow",
          "tags": [
            "nfdump"
          ],
          "dst.port": "53",
          "bytes": "61",
          "dst.ip": "x.x.x.x",
          "proto": "UDP",
          "src.ip": "y.y.y.y",
          "@timestamp":"2018-02-19T07:00:11.282Z",
          "src.as": "22222",
          "src.port": "20836"
        },
        "sort": [
          -61901113748481
        ]
      }
    ]
  }
}

In order to deserialize data into fields via encoding/json or encoding/xml, you need to make them public. E.g. this won't work:
type Flow struct {
	sas  int32    `json:"srs.as"`
	das int32     `json:"dst.as"`
	time time.Time `json:"@timestamp"`
}

but this will:
type Flow struct {
	Sas  int32    `json:"srs.as"`
	Das int32     `json:"dst.as"`
	Time time.Time `json:"@timestamp"`
}

If this isn't the issue, let me know and I'll reopen.


How to create 2d text other than html elements ?

is this possible right now? If it is, how?
Thanks!

Currently the TextMesh object supports 2D mode without 3D extrusion, i have also in development support for SDF text rendering wich allows for low memory, high quality font rendering for 2D fonts.
Thanks a lot!

Cool!
On Tue, Apr 9, 2019, 1:45 PM Tentone<EMAIL_ADDRESS>wrote:

Currently the TextMesh object supports 2D mode without 3D extrusion, i
have also in development support for SDF text rendering wich allows for low
memory, high quality font rendering for 2D fonts.
Thanks a lot!
[image: asasd]
https://user-images.githubusercontent.com/9158354/55822483-9b462c00-5af7-11e9-9f3e-a1fff2c83eb8.PNG
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tentone/nunuStudio/issues/291#issuecomment-481359743,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQLGlQSagElHgsXA2hnll6Tw4NXuZrhdks5vfNG0gaJpZM4cbS4o
.


Near-term roadmap

We're sharing a roadmap of ideas for improving and speeding up Gemma. If you'd like to join in and help us get there faster, please reach out so we can coordinate :)
Threading

[x] (jan-wassenberg) detect: total #logical, per-logical: package, chiplet, core, smt
[x] (jan-wassenberg) detect: CPU name, L2D/L3 size
[ ] (Z.A.) CCX-aware pinning
[ ] (jan-wassenberg) more efficient ThreadPool

Dot product

[x] Add _mm*_dpbf16_ps to HWY_AVX3_SPR and HWY_AVX3_ZEN4 targets, plus define HWY_NATIVE_DOT_BF16 in set_macros-inl.h
[x] Faster SFP decode via table lookup
[x] Add new NEON_* target that uses vbfdot for ReorderWidenMulAccumulate
[x] If !defined(HWY_NATIVE_DOT_BF16) || !HWY_NATIVE_DOT_BF16, decompress bf16->f32 to temp array before MatVec (idea by Samuel, thank you!) - in #166
[x] Apply even/odd trick to SFP
[ ] Convert f32 vectors to bf16 beforehand if HWY_NATIVE_DOT_BF16

Matmul

[x] (pculliton) implement basic matmul and test. Not using BLAS because we want to fuse matmul and decompression.
[ ] (szabadka, ...) Update Prefill to use matmul (activation @ weights) instead of MatVec
[ ] Cache-aware tiling

Compression

[ ] (pculliton, ...) Eval infrastructure
[ ] decide whether NUQ is launchable
[ ] improved compression, possibly error diffusion

General infra

[ ] (pculliton) Python wrapper
[ ] (pculliton, ...) Improved CI: run on Kaggle infra
[ ] AuxOut to hold timing info instead of printing in GenerateImpl.
[ ] Sampling struct holds rng and temperature, to reduce length of args

Is Paligemma part of the scope of gemma.cpp?

Let's discuss in #185 :)

Could you please confirm if this issue is resolved for you with the above comment ? Please feel free to close the issue if it is resolved ?
Thank you.


feat: rocks.nvim/luarocks support

Closes #14.
Things done:

Add a workflow that publishes tags to luarocks.org when a tag or release is pushed.

Add an scm rockspec that can be used to test luarocks installations locally, with luarocks make,
and to optionally upload dev releases to luarocks with luarocks upload --api-key <API_KEY> fugit2.scm-1.rockspec
Notes:

[!IMPORTANT]

For the luarocks workflow to work, someone with a luarocks.org account will have to add their API key to this repo's GitHub actions secrets.

Tagged releases are installed locally and then published to luarocks.org.

If you push tags from a local checkout, the workflow is triggered automatically.
If you use GitHub releases to create tags, you may need to add a PA token for the workflow to be triggered automatically.

[!TIP]

I recommend the release-please GitHub action to automatically generate release PRs based on conventional commits.
See also this guide.

When installing with luarocks, it will

Install all of the lua dependencies automatically.
Check for the presence of a libgit2 library, and fail if it is not found.

Due to a shortcoming in LuaRocks (https://github.com/luarocks/luarocks-site/issues/188), the neovim and/or vim labels have to be added  to the LuaRocks package manually (after the first upload), for this plugin to show up in https://luarocks.org/labels/neovim or https://luarocks.org/labels/vim, respectively.

@mrcjkb, some error with rockspec
Error: Could not load rockspec file /home/runner/work/fugit2.nvim/fugit2.nvim/fugit2.nvim-0.0.0-2.rockspec (Error running file: [string "/home/runner/work/fugit2.nvim/fugit2.nvim/f..."]:5: attempt to concatenate global 'specrev' (a nil value))
stack traceback:
	[C]: in function 'on_failure'
	...-luarocks-tag-release-scm-1/share/lua/5.1/ltr/os.lua:48: in function 'execute'
	...release-scm-1/share/lua/5.1/luarocks-tag-release.lua:77: in function 'create_rockspec'
	...release-scm-1/share/lua/5.1/luarocks-tag-release.lua:153: in function 'luarocks_tag_release'
	...ag-release/scm-1/bin/luarocks-tag-release-action.lua:84: in main chunk
	[C]: ?


(unknown) : Type not found : openfl.utils._internal.ExtraParamsMacro

Discussed in https://github.com/FNF-CNE-Devs/CodenameEngine/discussions/440

Originally posted by KadePleaseHelpMe November 11, 2024
Ran update.bat and then got this error. Solution?

That's on OpenFL's part. You could try deleting the local Haxe libraries folder first and then rerun the script; if it does not work, then I'm not sure what would be the correct solution for this.

Nex previously responded to this in discussions and it was the correct thing! Thank you anyway, but the problem is now fixed!


[BERT] Specify devices explicitly

This resolves device mismatch runtime failures when running on XLA.

Requires newly merged changes in swift-apis, which should resolve presubmit failures after tonight's nightly builds.


3015-Delete-user-profile-and-change-password - Add user deletion feature

3015-Delete-user-profile-and-change-password
Introduce functionality to deactivate users by setting their status to DELETED. Added corresponding methods in the service and controller layers, updated security configurations, and extended unit tests to cover this new feature.

Check sonarcloud issues


update engine-jupyter chart

renamed from spark-api to engine-jupyter.

use jupyter-spark Docker image and use --packages
to download engine, this seems to work with cluster.
mode whereas engine-jupyter Docker image does not.

make bblfshd optional.

updated bblfshd driver installation mechanism to be
the same as in bblfshd chart.

Signed-off-by: Santiago M. Mola<EMAIL_ADDRESS>

Closing, too outdated now.


[JENKINS-60940] - Migration of GitAPITests from JUnit 3 to JUnit 4

JENKINS-60940 - Migration of GitAPITests from JUnit 3 to JUnit 4
Added Class GitAPITest, for migration of GitAPITestCase from JUnit 3 to JUnit 4, added the first test getRemoteUrl
Checklist

[x] I have read the CONTRIBUTING doc
[x] I have referenced the Jira issue related to my changes in one or more commit messages
[x] I have added tests that verify my changes
[x] Unit tests pass locally with my changes
[x] I have added documentation as necessary
[x] No Javadoc warnings were introduced with my changes
[x] No spotbugs warnings were introduced with my changes
[x] I have interactively tested my changes

Types of changes
What types of changes does your code introduce?

[x] Infrastructure change (non-breaking change which updates dependencies or improves infrastructure)

This looks very good.  I'd like the pull request to include the removal of the test from GitAPITestCase so that it is clear in the single pull request that the test is moving from one location to another

@MarkEWaite  Sure, I will do that. And should I update more tests in this PR, or should I make another if in case you wish to merge this?

Thanks @MarkEWaite,
Small PR approach seems reasonable, it would help to avoid any big errors or failures, and tracking will be easy. I would follow that, as of now I have migrated smaller tests as suggested. Should I commit them for review?

Thanks @MarkEWaite,
Small PR approach seems reasonable, it would help to avoid any big errors or failures, and tracking will be easy. I would follow that, as of now I have migrated smaller tests as suggested. Should I commit them for review?

Yes, that would be great.  I've merged this change so that you have the base class available for subsequent pull requests.


ci-e2e: add e2e configurations for LRP

This PR adds the following configurations to cover LRP use cases.

socket-lb without kpr
per packet lb (aka socket-lb off)
socket-lb in hostns only + per packet

/test

/test

Is this a bugfix? Did Cilium 1.16 support implementing LRP via the packet path? I'm wondering just because this doesn't fit the obvious pattern for backporting to v1.16 per the docs guidance. But I may be wrong.

Yes, it is. The fix #33721 for the per packet mode which we are planning to backport to 1.16 doesn't work without this PR.

Hmm. That's making me wonder whether the other PR is also a bugfix or we're actually developing a new previously unsupported feature (per-packet LRP) that should be shipped in Cilium 1.17. Looking through #33721 I can see that I nominated the patch for backport to 1.16, but by now I don't understand why and I'm second-guessing whether the churn in code is low enough risk that it's worthwhile to backport to 1.16 or whether we should drop those labels and avoid the backport.

/test

/test


Testing Plugin

Hello,
I am testing your plugin and find this to be an awesome idea. I have a question in regards of the update option; what buffer do I use? The one created in indexeddb and how? Or one created somewhere else in memory/server? Could you please elaborate a bit more on this concept? Probably with an example of its use.
import {init} from '//unpkg.com/sqlite-worker';
init({name: 'my-db', update(uInt8Array) {
// store the latest uInt8Array somewhere
}});
My understanding is when the above code is run, it will grab same buffer used so I can continue doing crud operations on it. Am I right? I tried just like that but it does not seem to be doing anything at all.
Thank You.

the uInt8Array argument the update callback receives, is the entirety of the database as binary array.
that is the exact same binary array that would get stored into IndexedDB, hence a binary buffer usable by sqlite via emscripten directly, either on the server, or somewhere else.
the reason this possibility exists, is to be able to debounce, or synchronize on demand, the latest status of the database, in a remote server, or as a backup option.
does this help?

Hi Andrea,
Thank you for getting back to me this soon. It helps but still I have not been able to make it work. Let's say I want to use it as a backup; how do I do that? And let's say I want to retrieve that backed-up file and continue using it; how do I do it?
I tried this and still don't get see it doing that.
import {init} from '//unpkg.com/sqlite-worker';
init({name: 'my-db'}).then(async ({all, get, query}) => {
// ... same code as before ...
const {total} = await getSELECT COUNT(id) as total FROM todos;
//await queryDELETE FROM todos WHERE id=${5};
if (total < 1) {
console.log('Inserting some value');
//await queryINSERT INTO todos (id,value) VALUES (${4},${'d'});
}
console.log(await allSELECT * FROM todos);
init({name: 'my-db', update(uInt8Array) {
// store the latest uInt8Array somewhere
console.log("update/export");
}});
});
Regards,
C.Peretz

From: Andrea Giammarchi<EMAIL_ADDRESS>Sent: Wednesday, February 17, 2021 9:12 PM
To: WebReflection/sqlite-worker<EMAIL_ADDRESS>Cc: carper7<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [WebReflection/sqlite-worker] Testing Plugin (#6)
the uInt8Array argument the update callback receives, is the entirety of the database as binary array.
that is the exact same binary array that would get stored into IndexedDB, hence a binary buffer usable by sqlite via emscripten directly, either on the server, or somewhere else.
the reason this possibility exists, is to be able to debounce, or synchronize on demand, the latest status of the database, in a remote server, or as a backup option.
does this help?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/WebReflection/sqlite-worker/issues/6#issuecomment-780856110, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AELQMV3CGVLDTTVEHQ72223S7QWKZANCNFSM4XZCOWJA.

The example is clear though, and the readme too, you pass update as property of the direct init option:
init({name: 'my-db', update(uInt8Array) {
  // uInt8Array is *already* stored for you in the IndexedDB
  // but if you'd like to store it somewhere else you can
  console.log('last db length', uInt8Array.length);
}})
.then(async ({all, get, query}) => {
        // ... same code as before ...
       const {total} = await get`SELECT COUNT(id) as total FROM todos`;
      //await query`DELETE FROM todos WHERE id=${5}`;
      if (total < 1) {
            console.log('Inserting some value');
            //await query`INSERT INTO todos (id,value) VALUES (${4},${'d'})`;
      }
     console.log(await all`SELECT * FROM todos`);
  });

That array could be passed to the user as fallback in case there is no data anymore, as explained in the readme.
The "dance" is that if the user is known, but the user has no data, you can pass along a database file previously generated, so that previous data can be used... any better?
Also closing as this is not a bug per se.

P.S. the update is never called if no writing operations happen ... INSERT, DELETE, or UPDATE, are those that trigger updates and store in the IndexedDB, if you don't change anything, nothing happens.

Hi There,
Sorry to bug you too much.  This is not an issue but I cannot seem to get my head wrapped around sqlite-worker. I am trying to create a db and then store it in indexeddb; that part is ok but what I don't understand is what if I want to use this db everywhere in my js files. E.g. I want to perform an insert or delete a record; how do I do that;  it is not possible to use these:
import {init, SQLiteWorker} from '//unpkg.com/sqlite-worker';
import {init, SQLiteWorker} from './thirdparty/sqlite-worker/dist/index.js';
because imports are allowed from script tags with module type; so, I don't know how to use these methods once initialized.
I also saw somewhere in your github you use  importScripts but this does not work for me. I am sorry again to bother you it is just that I don't really know how to decouple all the components. I would like to be able to once I import all the required libs to just go and calling its methods from anywhere so I could perform all crud operations without a problem.
This is it. I want to thank you for any help you could  give and have a wonderful weekend.
Regards,
C.Peretz.

From: Andrea Giammarchi<EMAIL_ADDRESS>Sent: Thursday, February 18, 2021 6:04 AM
To: WebReflection/sqlite-worker<EMAIL_ADDRESS>Cc: carper7<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [WebReflection/sqlite-worker] Testing Plugin (#6)
P.S. the update is never called if no writing operations happen ... INSERT, DELETE, or UPDATE, are those that trigger updates and store in the IndexedDB, if you don't change anything, nothing happens.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/WebReflection/sqlite-worker/issues/6#issuecomment-781080425, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AELQMV6UI3XIN55FC6KM3OLS7SUWFANCNFSM4XZCOWJA.

I am trying to create a db and then store it in indexeddb

You don't need to do that, this library does that already.

what if I want to use this db everywhere in my js files.

all files need to pass through the DB handler ... you need to think ab out this like a remote DB, you have some API calling it and it's always pointing at 1 db.
You init the db more than once, you are already doing something wrong.
The API to query the DB is exposed once the DB is initialized:
https://github.com/WebReflection/sqlite-worker#importing-on-web-pages-via-esm
From that time on you can bootstrap any other script and pass along those utilities to query or provide a different API around those.

I know I am doing something wrong. The problem is I don't know exactly what; when I do it the same way you have it in your examples that works but when I try to do it the way I want it doesn't.
This works from html file within the script tags:

Now, this is what I am trying to do and it does not work:
HTML file:

JS file:
init({name: 'my-db', update(uInt8Array) {//enters here only when some sort of update to the db occurs
// store the latest uInt8Array somewhere
console.log('last db length', uInt8Array.length);
}})
.then(async ({all, get, query}) => {
// ... same code as before ...
await queryCREATE TABLE IF NOT EXISTS todos (id INTEGER PRIMARY KEY, value TEXT);
const {total} = await getSELECT COUNT(id) as total FROM todos;
await queryDELETE FROM todos WHERE id=${4};
if (total < 1) {
console.log('Inserting some value');
await queryINSERT INTO todos (value) VALUES (${'a'});
await queryINSERT INTO todos (value) VALUES (${'b'});
await queryINSERT INTO todos (value) VALUES (${'c'});
     }
     await query`INSERT INTO todos (id,value) VALUES (${4},${'d'})`;
     console.log(await all`SELECT * FROM todos`);
     await query`CREATE TABLE IF NOT EXISTS todos1 (id INTEGER PRIMARY KEY, value TEXT)`;
     console.log('Inserting some value');
    await query`INSERT INTO todos1 (value) VALUES (${'a1'})`;
    await query`INSERT INTO todos1 (value) VALUES (${'b1'})`;
    await query`INSERT INTO todos1 (value) VALUES (${'c1'})`;
    console.log(await all`SELECT * FROM todos1`);
    console.log(await all`SELECT * FROM todos1 WHERE id=2`);

});
Here is the error when using Api from JS file:
Uncaught TypeError: Cannot read property 'then' of undefined
at Object.onDeviceReady (formList.js:871)
at Object.bindEvents (formList.js:41)
at Object.initialize (formList.js:30)
at HTMLDocument.<anonymous> (formList.js:2610)
at j (jquery-1.11.2.min.js:2)
at Object.fireWith [as resolveWith] (jquery-1.11.2.min.js:2)
at Function.ready (jquery-1.11.2.min.js:2)
at HTMLDocument.J (jquery-1.11.2.min.js:2)

From: Andrea Giammarchi<EMAIL_ADDRESS>Sent: Saturday, February 20, 2021 8:22 AM
To: WebReflection/sqlite-worker<EMAIL_ADDRESS>Cc: carper7<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [WebReflection/sqlite-worker] Testing Plugin (#6)
I am trying to create a db and then store it in indexeddb
You don't need to do that, this library does that already.
what if I want to use this db everywhere in my js files.
all files need to pass through the DB handler ... you need to think ab out this like a remote DB, you have some API calling it and it's always pointing at 1 db.
You init the db more than once, you are already doing something wrong.
The API to query the DB is exposed once the DB is initialized:
https://github.com/WebReflection/sqlite-worker#importing-on-web-pages-via-esm
From that time on you can bootstrap any other script and pass along those utilities to query or provide a different API around those.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/WebReflection/sqlite-worker/issues/6#issuecomment-782585345, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AELQMVZYN4REC7HY24DFUF3S75WNBANCNFSM4XZCOWJA.

look, I've no idea what you are doing, the env is not clear, there's no live demo, the JS file means nothing, and I read jQuery within the stack of errors.
I don't now how you expect me to help ... but I won't reply anymore unless there is something concrete and reproducible I can test.


Remove empty columns in location graphbuilder tables COL-958

Interesting, I guess people are picky about showing data.  What about the csv download? This is probably not touched.

Agh - you make a damn good point about the downloads. I'm gonna see if it's super necessary.
Aside from that using the source to get the table map seems to work for everything but the industry / product profiles, and then there's some more complicated logic there because the source is the department / msa but the type of data is either industry or product or whatever. So I'm not sure how to tackle that in the long term.


Node.js

I cannot figure out how to validate the client certificate with a node.js app.
Does anyone know this?

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

ID: eb440603-950a-c224-f6c9-222ca502d5a5
Version Independent ID: 0b5c4944-0327-91fc-2b0b-c19a1b873b64
Content
Content Source
Service: app-service

I will assign this to author to enhance the document to include code sample to do client certificate validation for node.js app.
However, you may try below which could help unblock you for now.
If client auth certificate is created as a CA, then any certificate which is signed by this one will validate.
var options = {     key: fs.readFileSync('certs/server-key.pem'),     cert: fs.readFileSync('certs/server-crt.pem'),     ca: fs.readFileSync('certs/ca.pem'), #client auth ca OR cert     requestCert: true,                   #new     rejectUnauthorized: false            #new };
To handle validation -
app.use(function (req, res, next) {    if (!req.client.authorized) {        return res.status(401).send('User is not authorized');    }    #examine the cert itself, and even validate based on that!    var cert = req.socket.getPeerCertificate();    if (cert.subject) {        console.log(cert.subject.CN);    }    next(); });

This would be the way to do it when creating the server with https. But my tests with this didn't work, because I think Azure doesn't allow a nodejs https server. I want to validate it like stated in the documentation here: With the header 'X-Arr-ClientCert'.

Adding @cephalin as FYI

I've followed up with @naziml via email to see if we can get this addressed. Thanks for flagging this for us @janis91!

I will work with folks here on coming up with better docs on this. In the meantime, take a look at the code snippet on how to acquire the client cert from https://docs.microsoft.com/en-us/azure/app-service/app-service-web-configure-tls-mutual-auth. It's in C# but you should get the gist of what's needed there.

Is it possible to constraint the client certificate selection made by the user to a specific trusted certificate authority (CA) therefore filtering the client certs the user can choose from?

Closing due to inactivity.

@janis91 I've added a Node.js code sample per your contribution. Thank you!


install: place the library in architecture-specific directories on ELF platforms

This is needed for apple/swift#63782, which changes the Unix toolchain to look for libraries in architecture-specific directories.

Updated this pull to account for the recently merged #433, that also modified a few of these lines.

Closing since the Swift devs want to take a different approach.


adding join us page

@mmilenkovic could you check this pr as well that I can't assignee you because you did some changes too?

@mmilenkovic could you check this pr as well that I can't assignee you because you did some changes too?

Yes, sorry I missed this message.


Deleting a jar asset causes runtime problems

Recently when we renamed a jar to mariaDB4j-db-osx-2.1.9.jar, after Git import we get the following error on server startup.  Apparently we don't handle well the situation where a jar asset exists in the Archive but not in the current assets.
java.io.FileNotFoundException: /opt/apache-tomcat-8/sd-workflow/assets/com/centurylink/mdw/db/mariaDB4j-db-mac-2.1.9.jar (No such file or directory)
        at java.util.zip.ZipFile.open(Native Method)
        at java.util.zip.ZipFile.<init>(ZipFile.java:219)
        at java.util.zip.ZipFile.<init>(ZipFile.java:149)
        at java.util.jar.JarFile.<init>(JarFile.java:166)
        at java.util.jar.JarFile.<init>(JarFile.java:130)
        at com.centurylink.mdw.cloud.CloudClassLoader.findInJarFile(CloudClassLoader.java:205)
        at com.centurylink.mdw.cloud.CloudClassLoader.findInJarAssets(CloudClassLoader.java:240)
        at com.centurylink.mdw.cloud.CloudClassLoader.hasClass(CloudClassLoader.java:265)
        at com.centurylink.mdw.model.workflow.Package.getActivityImplementor(Package.java:507)
        at com.centurylink.mdw.services.process.ProcessExecutorImpl.prepareActivityInstance(ProcessExecutorImpl.java:720)
        at com.centurylink.mdw.services.process.ProcessExecutor.prepareActivityInstance(ProcessExecutor.java:230)
        at com.centurylink.mdw.services.process.ProcessEngineDriver.executeActivity(ProcessEngineDriver.java:332)
        at com.centurylink.mdw.services.process.ProcessEngineDriver.processEvent(ProcessEngineDriver.java:612)
        at com.centurylink.mdw.services.process.ProcessEngineDriver.executeServiceProcess(ProcessEngineDriver.java:830)
        at com.centurylink.mdw.services.process.ProcessEngineDriver.invokeService(ProcessEngineDriver.java:741)
        at com.centurylink.mdw.services.process.ProcessEngineDriver.invokeService(ProcessEngineDriver.java:679)
        at com.centurylink.mdw.services.workflow.WorkflowServicesImpl.invokeServiceProcess(WorkflowServicesImpl.java:913)

I think the issue here was that we were caching the list of JARs (per CloudClassLoader object instance), so if the file changed names, CloudClassLoader doesn't know about it.   I have moved the "caching" part into AssetCache class so that it can be re-generated when a cache refresh takes place.
Is there an easy way to test this?

Ok, so issue was that we were building the path to the asset by concatenating assetRoot + "/" + jarAsset.getPackageName().replace('.', '/') + "/" + jarAsset.getName(), which won't work on any JAR file located in Archive folder since the path will always resolve to the path of the latest pkg, which means it would never search JARs in Archive sub folders.    I'm fixing this by making use of the Asset.getRawFile() method to retrieve the actual JAR file instead of trying to build a new File with a derived concatenated path.


origin pool update in terraform ERROR: does not correlate with any element in actual

I have an existing Volterra Azure setup with HTTP lb and origin pool. I performed the following actions:

Logged into azure portal
deleted webserver virtual machine
terraform apply

I then re-ran my terraform apply to rebuild resources. The origin pool needs to be updated, but terraform gave an error:
Error: Provider produced inconsistent final plan

When expanding the plan for module.azure[0].volterra_origin_pool.app["bu12"]
to include new values learned so far during apply, provider
"registry.terraform.io/volterraedge/volterra" produced an invalid new value
for .origin_servers[0].private_ip: planned set element
cty.ObjectVal(map[string]cty.Value{"inside_network":cty.True,
"ip":cty.UnknownVal(cty.String), "outside_network":cty.NullVal(cty.Bool),
"site_locator":cty.SetVal([]cty.Value{cty.ObjectVal(map[string]cty.Value{"site":cty.SetVal([]cty.Value{cty.ObjectVal(map[string]cty.Value{"kind":cty.UnknownVal(cty.String),
"name":cty.StringVal("giroux-mcn-bu12-azure-262d"),
"namespace":cty.StringVal("system"),
"tenant":cty.StringVal("f5-gsa-dhqmwfao")})}),
"virtual_site":cty.SetValEmpty(cty.Object(map[string]cty.Type{"kind":cty.String,
"name":cty.String, "namespace":cty.String, "tenant":cty.String}))})})}) does
not correlate with any element in actual.

This is a bug in the provider, which should be reported in the provider's own
issue tracker.

Quick update, I simply re-applied terraform (no changes made) and no errors this time on the remaining origin pool edit. The object was originally updated successfully based on what I see. Not sure why I got this error. Deployed this same repo tons of times and this is the first time I hit it. However, this is the first time I did a backend VM delete and apply to origin pool for changes. Though, the IP of the backend webserver remained <IP_ADDRESS> the entire time, so really...no changes were needed or performed on the volterra side.


Renaming strings to match the new glossary.

@jansorg
PLEASE DO NOT MERGE
No new functionality, just string changes!

Could you also help me out in figuring out why the test is failing?

@mwichary Travis' tests are sometime unreliable, so if you see a grey build or an error related to vscode install/npm install it's safer to restart that build to make sure it's not just an issue with the whole build stuff. In our case it was that, now that the error is expected '' to sort of equal 'Jump to  How to' in test/assets/jump-to.test.js:72:38.


Release Google.Cloud.Kms.V1 version 3.1.0

Changes in this release:
New features

Enable generation of Locations mixin (commit 4475616)

Created release for Google.Cloud.Kms.V1-3.1.0


PHP 7.0 Support

So i know the NewRelic doesn't officially support PHP 7.0 yet but 1 change that will need to occur is the php5enmod command. With php 7.0 it has changed to phpenmod.
The same goes for the php directory. It has also dropped the 5 in favor of just php.
One way this could be fixed is letting users specify the phpenmod command to run. Currently it is a bool to run the command but it could be changed to a string that can be nil. So if a user needs to run phpenmod they would just need to specify the command instead of nil.

Hi @bassrock,
what about perhaps checking the installed version of PHP and use that do drive the command's value?
Thanks in advance!
Kind regards,
David

I made this PR for add PHP 7.0 support https://github.com/djoos-cookbooks/newrelic/pull/268


maxima: 5.42.2 -> 5.44.0

Motivation for this change
This change updates Maxima to the latest release. The change logs for 5.43 and 5.44 are available.
$ nix path-info -S nixpkgs#maxima
/nix/store/n3j4fqwypjayckkkjvc7akw4hilagj41-maxima-5.42.2         275446544
$ nix path-info -S ./result
/nix/store/nf8i698s6dhv0z9dp8aph92rgyd0hbd1-maxima-5.44.0         278350072

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Sage has unfortunately been broken on master for a while (https://github.com/NixOS/nixpkgs/pull/70474#issuecomment-712666618 for context). @omasanori is currently working on fixing it up, so they have probably taken the implications on sage into account.

potential regression with sage

@omasanori is currently working on fixing it up, so they have probably taken the implications on sage into account.

Yes, 5.44.0 is the bundled version with the latest (to-be-packaged-in-nixpkgs) SageMath.

Addressed to @veprbl's comment.


How can I set mounted secret permissions?

Guys,
Need a little help here: How can I set mounted secret permission?
With EKS secret it's set via
   secret:
     defaultMode: 0600

with result:
-rw-------    1 root     root          1864 May  2 22:01 .mylogin.cnf
With AWS secret it's:
-rw-r--r--    1 root     root          1864 May  2 22:18 .mylogin.cnf
What parameter controls it?
How do I set it to be -rw-------
The pod yaml is:
apiVersion: v1
kind: Pod
metadata:
  name: busybox
spec:
  serviceAccountName: tester
  containers:
  - name: busybox
    image: registry.k8s.io/busybox:latest
    command: ["sh"]
    args:
    - -c
    - ls -la /root/
    volumeMounts:
    - name: mylogin
      mountPath: /root/.mylogin.cnf
      subPath: .mylogin.cnf
      readOnly: true
  restartPolicy: Never
  volumes:
  - name: mylogin
    csi:
      driver: secrets-store.csi.k8s.io
      defaultMode: 0600
      readOnly: true
      volumeAttributes:
        secretProviderClass: "aws.mylogin.cnf"
  # volumes:
  # - name: mylogin
  #   secret:
  #     defaultMode: 0600
  #     secretName: mysql
  #     items:
  #     - key: .mylogin.cnf
  #       path: .mylogin.cnf

This value is coming from the upstream CSI Driver. We will reach out to the team to see if there are any recommendations.

Upstream permissions gets un-marshaled here and has test coverage.


Add ability to configure matmul tilings from the Python API

This allows manually overriding the tiling configuration for the MatMul
kernel through parameters in MatMulP and exposes that configuration
through the C and Python API.
In addition, a new Python example script is included which runs an
optimal candidate search from Python with a couple options (e.g. kernel
and device selection) which can be used as a starting point for further
experiments.

bors r+

bors retry

My bad, I gave you the wrong code. Its .map(|x| vec![x]) instead of .map(Vec::new).

bors retry

bors r+


Why Many TSXXXX ERROR in UI Front-end construction?

I just download source code,and run:
make yarn_install_cache
make yarn_start_dev_ui
so many error displayed:
ERROR in src/utils/ServiceUtils.tsx:209:12
TS2746: This JSX tag's 'children' prop expects a single child of type 'ReactNode', but multiple children were provided.

already fixed，remove node_modules，and build without error.


Milestone 2, January Release 2020 - Build Triage (Failures)

JDK8 : https://trss.adoptopenjdk.net/resultGrid?parentId=5e17394b291346186001b563

Failure
Platform:
ppc64le_linux
ppc64le_linux_xl
s390x_linux
s390x_linux_xl
x86-32_windows
x86-64_linux
x86-64_linux_xl
x86-64_mac
x86-64_mac_xl
x86-64_windows
x86-64_windows_xl

Details:
sanity.openjdk failures
java/lang/Class/getMethods/StarInheritance.java.StarInheritance (from java_lang_Class_getMethods_StarInheritance.java)

The issue has been raised https://github.com/eclipse/openj9/issues/8255 - Closed
And Still in progress under https://github.com/eclipse/openj9/issues/7623.

Failure

Platform
x86-64_linux
Details
special.functional failures

Test name : openjdk-tests/functional/JIT_Test
/home/jenkins/workspace/Test_openjdk8_j9_special.functional_x86-64_linux/openjdk-tests/functional/JIT_Test 15:05:24 autoGen.mk:758:  Malformed target-specific variable

Issue Raised: https://github.com/AdoptOpenJDK/openjdk-tests/issues/1540
Reported as Machine issue, ran absolutely fine in another machine. : Closed

Failure

Platform
x86-64_mac
x86-64_mac_xl
Detail
special.openjdk failure
jdk_math_jre_0
Issue raised: https://github.com/eclipse/openj9/issues/8271 : Closed
The fix is under https://github.com/AdoptOpenJDK/openjdk-tests/issues/1525 : Closed

JDK11 : https://trss.adoptopenjdk.net/resultGrid?parentId=5e1735c3291346186001a686

Failure

Platform
s390x_linux_xl
Details
santy.openjdk failures

34 Test failures
sun/nio/cs/FindDecoderBugs.java.FindDecoderBugs
sun/nio/cs/FindOneCharEncoderBugs.java.FindOneCharEncoderBugs
sun/nio/cs/ISCIITest.java.ISCIITest
sun/nio/cs/ISO8859x.java.ISO8859x
sun/nio/cs/JISAutoDetectTest.java.JISAutoDetectTest
sun/nio/cs/LatinCharReplacementTWTest.java.LatinCharReplacementTWTest
sun/nio/cs/LeftOverSurrogate.java.LeftOverSurrogate
sun/nio/cs/NIOJISAutoDetectTest.java.NIOJISAutoDetectTest
sun/nio/cs/SJISCanEncode.java.SJISCanEncode
sun/nio/cs/StreamEncoderClose.java.StreamEncoderClose
sun/nio/cs/SurrogateGB18030Test.java.SurrogateGB18030Test
sun/nio/cs/SurrogateTestEUCTW.java.SurrogateTestEUCTW
sun/nio/cs/SurrogateTestHKSCS.java.SurrogateTestHKSCS
sun/nio/cs/Test4206507.java.Test4206507
sun/nio/cs/TestConverterDroppedCharacters.java.TestConverterDroppedCharacters
sun/nio/cs/TestCp834_SBCS.java.TestCp834_SBCS
sun/nio/cs/TestCp93xSISO.java.TestCp93xSISO
sun/nio/cs/TestEBCDICLineFeed.java.TestEBCDICLineFeed
sun/nio/cs/TestEUC_TW.java.TestEUC_TW
sun/nio/cs/TestGB18030.java.TestGB18030
sun/nio/cs/TestHKSCS.java.TestHKSCS
sun/nio/cs/TestIBMBugs.java.TestIBMBugs
sun/nio/cs/TestISO2022CNDecoder.java.TestISO2022CNDecoder
sun/nio/cs/TestISO2022JP.java.TestISO2022JP
sun/nio/cs/TestISO2022JPEncoder.java.TestISO2022JPEncoder
sun/nio/cs/TestISO2022JPSubBytes.java.TestISO2022JPSubBytes
sun/nio/cs/TestIllegalISO2022Esc.java.TestIllegalISO2022Esc
sun/nio/cs/TestIllegalSJIS.java.TestIllegalSJIS
sun/nio/cs/TestJIS0208Decoder.java.TestJIS0208Decoder
sun/nio/cs/TestJIS0212Decoder.java.TestJIS0212Decoder
sun/nio/cs/TestMS5022X.java.TestMS5022X
sun/nio/cs/TestMiscEUC_JP.java.TestMiscEUC_JP
sun/nio/cs/TestSJIS0213.java.TestSJIS0213

Rerunning the grinder job: https://ci.adoptopenjdk.net/job/Grinder/1683/ : Passed

Failure

Platform
x86-64_windows_xl
Detail
sanity.openjdk failures
Not able to clear the directory

Looks like the build machine issue- failed in clearing the workspace. Request the Adopt Infrastructure team to rebuild this job again to see if that clears the issue.
From Infra team: The machine has been reboot which should clear this issue.
Issue Raised : https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1088 : Closed

JDK13 : https://trss.adoptopenjdk.net/resultGrid?parentId=5e177c1e291346186001d1c5

Failure

Platform
All supported platforms
Details
sanity.perf is failing.
Console log from one of the platform: https://ci.adoptopenjdk.net/job/Test_openjdk13_j9_sanity.perf_ppc64le_linux/4/console
The issue has been raised https://github.com/AdoptOpenJDK/openjdk-tests/issues/1541 : Closed (Issue has been addressed and fixed)

Failure

Platform
x86-64_linux
Detail
sanity.openjdk is failing

java/net/Socket/LinkLocal.java.LinkLocal
java/net/httpclient/DependentPromiseActionsTest.java.DependentPromiseActionsTest

Rerun the grinder job https://ci.adoptopenjdk.net/job/Grinder/1685/ : Passed

Failure

Platform
x86-64_windows
Detail
https://ci.adoptopenjdk.net/job/Test_openjdk13_j9_sanity.system_x86-64_windows/106
sanity.openjdk and sanity.system
Looks like the build machine issue - failed in clearing the workspace. Request the Adopt Infrastructure team to rebuild this job again to see if that clears the issue.
AdoptOpenJDk Infrastructure issue has been raised :
https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1086
Rerun the job in diff machine: https://ci.adoptopenjdk.net/job/Test_openjdk13_j9_sanity.openjdk_x86-64_windows/107/ : Passed

Failure

Platform
x86-64_windows_xl
Details
extended.system failures
Issue with playlist.xml: The entity name must immediately follow the '&' in the entity reference.

Issue raised : https://github.com/AdoptOpenJDK/openjdk-tests/issues/1539 :   Closed


Plan for es3.d.ts

lib.d.ts for es3, lib.es5.full.d.ts for es5
fourslash baselines are for lib.d.ts, give fourslash test lib: es3 by default to suppress baseline changes as much as possible
support @lib setting on fourslash test to allow setting @lib: es5 for ES5-requiring tests
add a test to check @lib: es3 does not have ES5 types

https://github.com/microsoft/TypeScript/pull/28372 is rejected


Can this be more web designer friendly?

As a UX person who spends some time working on the front-end, I think some of these items on the list would have big implications on the way designers work, like accessibility and such, but I'm not sure... Our community is rather uneducated on these matters. Are there items on this list you can make more designer-friendly, like maybe drilling down the outline more, or including resources?

Thanks @mallory-box!
I agree that some of these could have the potential to impact how a designer works. The more I've thought about it, I see so many correlations between these ideas and the field of UX. This site is based on the content of a series of free digital books that will be published by O'Reilly over the next year or so. I've only just started the (messy) writing process, so I haven't gotten too deep yet, but the content will definitely be expanded and include resources. I'll also plan to expand this site as I continue to write.
All of that said, I'm not a designer and would really appreciate contributions and feedback from designers! I plan on soliciting feedback from my design friends and colleagues, but would love more from a wider group. So, if these things interest you or any designers you may know, please feel encouraged to provide feedback or even contribute/improve content. You can do that through either of these GitHub repos, Twitter , or even email.
Thanks for reaching out, it means a lot!


Silence the detachedHead warning when ref_type is a revision

The PR that added this #8622 didn't actually resolve the problem because you see the warning when the @ref_type you've specified is a :revision and you haven't specified a :branch or :tag.
The output looks like this:
==> Checking out revision 3b3527f5050270cf1acd280691b09b5abe130e73
Note: switching to '3b3527f5050270cf1acd280691b09b5abe130e73'

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

   git switch -c <new-branch-name>

Or undo this operation with:

   git switch -

Turn off this advice by setting config variable advice.detachedHead to false

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Makes sense to me, but what scenario does that extra part in config_repo cover?

The extra part in config_repo covers the scenario where you're upgrading a formula (that uses :revision) that you'd previously installed. If you already have a repo cloned to brew --cache -s <that-formula>, then when you checkout a new revision, git would show that detached head message again unless we reclone it or just set that config in config_repo

As in a repo cached before the change? Since -c advice.detachedHead=false will automatically set git config for new clones.

I'm fine with this when @Bo98 is.

As in a repo cached before the change?

Yep! Exactly

Thanks!


[GHSA-9c47-m6qq-7p4h] Prototype Pollution in JSON5 via Parse Method

Updates

Affected products

Comments
According to the 1.0.2 release notes [1] the bugfix has been backported to v1.
This has also been updated on the Security Advisory published to the json5 repo [2], but for some reason not here.
[1] https://github.com/json5/json5/releases/tag/v1.0.2
[2] https://github.com/json5/json5/security/advisories/GHSA-9c47-m6qq-7p4h

Hi there @jdgregson and @jordanbtucker! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

Duplicate.


SwapChainPanel has a thin hole on the sides

Describe the bug
SwapChainPanel has a thin hole on the sides. The hole pierces through the complete WinUI rendering layer, the color visible underneath comes from GCLP_HBRBACKGROUND window class style.
Steps to reproduce the bug
Create a desktop WinUI3 app that uses SwapChainPanel to render something. The SwapChainPanel needs a working IDXGISwapChain to trigger the bug. It doesn't matter what you render in the swap chain.
Expected Behavior
SwapChainPanel should blend the content on top of WinUI’s visual elements behind the panel. If should never blend the content on Win32 background of the window.
Screenshots
For better visibility of the bug, I've called CreateSolidBrush of color 0xFF00FF, and called SetClassLongPtr( hwnd, GCLP_HBRBACKGROUND, hbrush );

If you wonder why the top area of the background is different — because the main window sets ExtendsContentIntoTitleBar = true; that area comes from the title bar behind WinUI visuals.
Version Info
Windows 10 20H2 19042.985, WinUI 3 packages version 0.5.7

@codendone FYI

This should be fixed in version 0.8. Please reactivate if you still repro  on 0.8.x.


Open port 8080 and 22 outbound for OHW

Ref https://github.com/2i2c-org/pilot-hubs/issues/549#issuecomment-892276020

Merging since this is already deployed


"Invalid 'main' field" warning

I get the following warning when running the action:
(node:4284) [DEP0128] DeprecationWarning: Invalid 'main' field in '/home/runner/work/_actions/azure/webapps-deploy/016bdd3f9b7cec60310bcf9da98f671628795644/node_modules/actions-secret-parser/package.json' of 'lib/index.js'. Please either fix that or report it to the module author

Had the same issue when trying to deplay an ASP.NET 6 api. Problem was solved by updating actions/checkout@v2 and actions/setup-dotnet@v1 to actions/checkout@v3 and actions/setup-dotnet@v3

Had same issue now when using:
uses: actions/checkout@v3
uses: actions/setup-node@v3
uses: actions/upload-artifact@v3
uses: actions/download-artifact@v3

(node:2310) [DEP0128] DeprecationWarning: Invalid 'main' field in '/home/runner/work/_actions/azure/webapps-deploy/v2/node_modules/actions-secret-parser/package.json' of 'lib/index.js'. Please either fix that or report it to the module author
(Use node --trace-deprecation ... to show where the warning was created)
Package deployment using ZIP Deploy initiated.

My full workflow file is below incase it helps with troubleshooting:
# Docs for the Azure Web Apps Deploy action: https://github.com/Azure/webapps-deploy
# More GitHub Actions for Azure: https://github.com/Azure/actions

name: Build and deploy Node.js app to Azure Web App - <app-name-here>

on:
  push:
    branches:
      - main
  workflow_dispatch:

jobs:
  build:
    runs-on: ubuntu-latest

    steps:
      - uses: actions/checkout@v3

      - name: Set up Node.js version
        uses: actions/setup-node@v3
        with:
          node-version: '18.x'

      - name: npm install, build, and test
        run: |
          npm install
          npm run build --if-present
          npm run test --if-present

      - name: Upload artifact for deployment job
        uses: actions/upload-artifact@v3
        with:
          name: node-app
          path: .

  deploy:
    runs-on: ubuntu-latest
    needs: build
    environment:
      name: 'Production'
      url: ${{ steps.deploy-to-webapp.outputs.webapp-url }}

    steps:
      - name: Download artifact from build job
        uses: actions/download-artifact@v3
        with:
          name: node-app

      - name: 'Deploy to Azure Web App'
        id: deploy-to-webapp
        uses: azure/webapps-deploy@v2
        with:
          app-name: '<app-name-here>'
          slot-name: 'Production'
          publish-profile: ${{ secrets.AZUREAPPSERVICE_PUBLISHPROFILE_<********> }}
          package: .

This has been fixed as part of #280, will be rolled out in next release.


[WIP] using secp256k1-node v3.0.0

Some optimizations with secp256k1-node v3.0.0 (hope will be available soon).

I totally don't understand verifySignature and validateSin callback pattern. Why not just throw error or return boolean? All operations are synchronous.
Why you don't use ECIES? With integrated encryption BitAuth can be used with HTTP.

Updated!

Interesting that the bitauth-browserify.js and bitauth-node.js is now longer needed.

@fanatid I was looking at this last night in attempt to get bitauth updated to v3.2.2 of the bindings. Most of it looks pretty good, but there is one problem.
secp256k1-node now returns false when verifying signatures with high s values. This is a problem for us. The bitauth spec does not require the strict signature style found in BIP0062.
We have BitPay api clients in a number of different languages. I looked at a few, (C#, ruby, php), and they don't do any kind of s value checking. If we started verifying signatures with the new bindings, lots of api requests to bitpay.com would fail signature validation.
Is it possible to add an additional api to the bindings to do non-strict signature verification? I would prefer not to have to normalize all signatures in order to verify them.
As I understand it, we would also want to use secp256k1..signatureImportLax() instead of secp256k1..signatureImport() in order to handle non-DER signatures.

@gabegattis it is possible add additional api method, but I do not want to do this, because bitcoin-core/secp256k1 doesn't  have this and as result I will have to make something that did with ecdh.
So answer is no, we should use normalize and then verify :)

Ok. I will guess I will need to do some testing with normalization. Do you have any concept of the computational cost for normalizing/verifying a signature compared to just verifying an already-normalized sig? I will make some benchmarks at some point.

@gabegattis normalize is very-very fast:
const bindings = require('./bindings')

const priv = Buffer.from('c26848a3a9b8c60ba6757d6e3c8baa0a0ded80887510b4eebc0980db74415684', 'hex')
const pub = bindings.publicKeyCreate(priv)
const msg = Buffer.from('3dc81fd01b4b8b23e1c92cb461b5752d76bad4d990b9f4d591ff7fc77d4b8419', 'hex')
const sig = bindings.sign(msg, priv).signature

console.time('verify')
for (let i = 0; i < 10000; ++i) { bindings.verify(msg, sig, pub) }
console.timeEnd('verify')

console.time('normialize')
for (let i = 0; i < 10000; ++i) { bindings.signatureNormalize(sig) }
console.timeEnd('normialize')

on my machine:
verify: 1936.436ms
normialize: 20.875ms

verify -- ~5000 ops/s
normalize -- ~5000000 ops/s

That is good news then. I just tried my own benchmark as well, but using a high s value sig. I got similar results. Normalizing is about 100 times faster than verifying.


Allow user to set CONNECT_API_SERVER

part of

#93

Yes, I believe it is CONNECT_SERVER (that's what is in my env variables). 👍

Looks like board_rsconnect started reading the CONNECT_SERVER variable at some point..!
https://github.com/rstudio/pins-python/blob/main/pins/constructors.py#L395


Migrating old endpoints to v2 API

Description
after merging #283 , the old code is still working (old controllers/services), we need to migrate them to the new code and expose the APIs under /v2, above are the tasks needed to accomplish this:
Check List

[ ] Call /v2/Contributors endpoint from frontend, instead of old endpoint /Contributors, and remove the last.

The API code for Contributors Controller get method is here
The Frontend code for calling the old endpoint is here and here

[ ] Create /v2/GithubUsers endpoint and call it from frontend instead of /github/users, and remove the last.

You will create a new method inside Github service similar to listContributors method, but the logic will be that of the old getUser method
You will create a new module api/src/github-user and inside you create your controller api/src/github-user/controller.ts similar to Contributors' controller. where you call the newly created method of GithubService.
Then update the frontend to call the new endpoint instead of the old one here and here

Additional Comments
please create new PR for each task.

@well-balanced this is much clearer task i believe, if you wanna take up any, feel free to do so, and ofc i'm here for any questions :)

@ZibanPirate thanks for your clean description I really appreciate it 🙏 I've just implemented tasks those are included in the checklist. check plz 😊

checking ...

closing as done in #290 and #289


xSQLServerSetup: Installation hangs without any error when using UNC path

Installation hangs without any error. This happens when using UNC path in the SourcePath parameter and at the same time is not using SourceCredential.
After some debugging, it seems that Start-Process hangs when calling setup.exe and does not return.
This will be solved when PR #123 is merged.
I think the culprit is security. ;) This one that popup when I ran the command
Start-Process -FilePath \\fs.company.local\images\SQL2016RTM-ENT\setup.exe
    -ArgumentList '/IAcceptSQLServerLicenseTerms="True" /Action="Install" /InstanceName="SQL2016" /UpdateEnabled="False" /Features="IS" /BrowserSvcStartupType="Automatic"'

You can get around this by adding the domain to the trusted zone Local Intranet. But that is only for the logged on user. So won't do it for DSC.

Tried changing the local group policy to allow .exe. But that did not work. Did not try setting this in the domain, since it opens up to much.
Local Computer Policy/User Configuration/Administrative Templates/Windows Components/Attachment Manager

Setting: Inclusion list for low file types
Value: .exe

What did work was to run Start-Process with the -NoNewWindow parameter. It then use the Microsoft .NET Framework CasPol (CasPol.exe) to "override" this lite bump. Don't know any more than that. If someone know more what happens here, please enlighten me 😄
PS C:\Windows\system32> Start-Process -FilePath \\fs.company.local\images\SQL2016RTM-ENT\setup.exe
    -ArgumentList '/IAcceptSQLServerLicenseTerms="True" /Action="Install" /InstanceName="SQL2016" /UpdateEnabled="False" /Features="IS" /BrowserSvcStartupType="Automatic"' -NoNewWindow

Microsoft (R) SQL Server 2016 13.00.1601.05
Copyright (c) 2016 Microsoft Corporation.  All rights reserved.

Microsoft .NET Framework CasPol 4.0.30319.33440
for Microsoft .NET Framework version 4.0.30319.33440
Copyright (C) Microsoft Corporation.  All rights reserved.

WARNING: The .NET Framework does not apply CAS policy by default. Any settings shown or modified by CasPol will only
affect applications that opt into using CAS policy.

Please see http://go.microsoft.com/fwlink/?LinkId=131738 for more information.

Success
Microsoft .NET Framework CasPol 4.0.30319.33440
for Microsoft .NET Framework version 4.0.30319.33440
Copyright (C) Microsoft Corporation.  All rights reserved.

WARNING: The .NET Framework does not apply CAS policy by default. Any settings shown or modified by CasPol will only
affect applications that opt into using CAS policy.

Please see http://go.microsoft.com/fwlink/?LinkId=131738 for more information.

Success
SQL Server 2016 transmits information about your installation experience, as well as other usage and performance data, t
o Microsoft to help improve the product. To learn more about SQL Server 2016 data processing and privacy controls, please see the Privacy Statement.

Found this that explains more about CasPol.
Caspol.exe (Code Access Security Policy Tool)
Using CasPol to Fully Trust a Share
This would trust everything under the 'foldershare'.
CasPol.exe -polchgprompt off -machine -addgroup 1.2 -url \fs.company.local\foldershare* FullTrust
So then Start-Process must call CasPol to trust the path in the -Path parameter when the parameter `-NoNewWindow' is used.

For reference. This error occurs when not using the parameter SourceCredential.

In PR #578 there is a new timeout parameter (SetupProcessTimeout) which is meant to partly resolve this by at lest throwing an error if the setup is not finished within 2 hours (default value).

With PR #578 I cannot reproduce this one. So I think this get's solved in PR #578 thanks to it now starts the process using Setup-Process.


Fixed #25211 -- Add USE_X_FORWARDED_PORT setting and HttpRequest.get_port()

It's be nice to internally have a way to reference the "real" port number when behind a proxy.
Today, as an example:
If you were running a load balancer on port 80, which proxied back to an nginx which was running on 8080, it's possible that get_host() would resolve your request back to example.com:8080 instead of the proper example.com since SERVER_PORT is coming from what nginx was listening on.
This just adds a proper method to get_port() and respect an optional X-Forwarded-Port header.
The benefits of this being internal to Django, is because anywhere that Django tries to resolve the port internally, we should be utilizing this method. This issue is easy to correct in user-land, but a bit harder when an internal Django method utilizes SERVER_PORT directly. The only option is to have a middleware which sets request.META['SERVER_PORT'] = request.META['HTTP_X_FORWARDED_PORT']` which is less than ideal IMO since it's overloading the meaning of the variable.
This is also relevant for #4337 since in the wild, we hit an issue with SERVER_PORT not reflecting what the upstream actually was, which caused issues. So this would unify and make #4337 leverage the new get_port().

Looks pretty good!

btw, the test/lint failure look unrelated. Not sure how to re-trigger a build.

@mattrobenolt please rebase against latest master. The default failure seems moot and the flake8 has been fixed in fa14fb9771a63c33a904f84db6027f4b1ea2b33f.

@charettes :pray:

@mattrobenolt, code and docs are looking good.
Please open a new ticket to track this feature addition request and mark it as accepted with a patch.

Done. https://code.djangoproject.com/ticket/25211

Marked the patch RFC pending a small addition to the 1.9 release notes.

Into docs/releases/1.9.txt ?

Added to release notes.

merged in 4dcfbd792342667174834c40e792f1b8ac2f0609, thanks!

@timgraham @charettes @aaugustin thx for the fast feedback and review. :heart:


Revising Metric: Issues Closed

Name of the Metric Being Revised

Issues Closed

Website: https://chaoss.community/metric-issues-closed/
File: https://github.com/chaoss/wg-evolution/blob/main/focus-areas/issue-resolution/issues-closed.md

Link to Original Metric Release Issue

#288
Specific Metric Details that Require Attention

As in the other Issues metrics, there are specific instructions for implementing this metric in GrimoireLab, so we should verify these are accurate or remove them to keep from having to check this in the future. Also, no mention of Augur.
The objectives section could use some work
There are parameters which seem like filters to me
Synonyms are in "Data Collection Strategies" and not the Description section
Do we need references?
This is one of our more heavily referenced metrics, so we should make sure it's :sparkles:

Checklist

Process

[x] Create the “revising metric” labeled issue in the authoring WG’s repo for comments during review period
[ ] Create pull request of revised metric to WG’s repo (after checking Content Quality and Technical Requirements below)
[ ] Add the metric revision to release notes issue in working group repo
[ ] Update the Metrics Spreadsheet to indicate that the revised metric is under review
[ ] Create issue in CHAOSS/Translations repository to kick-off translation to other languages (please use the the translation issue template)
[ ] "Metric Candidate Release" label added to the metric release candidate issue when fully ready fo release

When above steps are completed:

[ ] Announce new/updated metric on mailing list, newsletter, community Zoom call, and Twitter. This can be coordinated with the community manager and can be done as a collective of all revised metrics from the CHAOSS community.

Content Quality (check all that apply)

[ ] Date of last review has been added to the revised metric at the bottom of the markdown file (month/year)
[ ] Formatting changes have been made
[ ] Minor editorial changes have been made
[ ] Major editorial changes have been made
[ ] Metric adheres to the current metrics template
[ ] Metric name has changed and necessary updates are made to the WG README table and metrics tracking spreadsheet

Technical Requirements for any Revisions

[ ] Metric file name is the full metric name and only contains lower case letters and hyphens (“-”) for spaces
[ ] Images are included using markdown and absolute path links (as described in the metrics template)
[ ] Images have at least one empty line above and below them
[ ] Images are placed in image folder and followed naming convention
[ ] If new a focus area is created, ensure focus area is added to wg repo readme and focus area folder readme
[ ] There is no HTML code in the metrics markdown file

Note the "Specific Metric Details that Require Attention" section above


Terminal is crashing if we drag a tab and try to reorder

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
 Win32NT             10.0.18363.0 Microsoft Windows NT 10.0.18363.0
Windows Terminal version (if applicable): preview

Any other software?

Steps to reproduce

Open Windows terminal with more than one tabs
try reorder tabs by drag and drop

Expected behavior

The tab which is getting dragged and dropped should be shifted to dropped position
Actual behavior

Terminal is crashing.

@upadhyayap Are you running elevated / as Administrator? If so this'll be #3581

yes i am running as elevated. I see this is being tracked #3581

@zadjii-msft can we disable drag&drop completely? this is very confusing and annoying.

As mentioned in #3581, our plan is to:

detect when running as administrator and disable drag & drop activity.


Filtering

Very nice calendar. Do you plan to support filtering of events at the calendar level and view all events? I see search highlights the event on a page, but looking across the entire calendar or analyzing a particular criteria would be helpful.

Hi,
This is coming in v1.4.0.
Thanks

Release today.  Enjoy!
Thanks


docs: new landing page design

New landing page design

@mowies

@thisisobate please check how you like the new graphics :)
also dark mode definitely still needs some work

mobile version also needs some work still 😅

could we change the text at the top, gradient on gradient is hard to read

oh yes, @mowies is taking a look at this. please note that this issue only applies to dark theme

Deploy preview

@thisisobate @nate-double-u we made some updates to dark mode, pls have a look one more time. We are happy with it now :)

@thisisobate one more thing on mobile: it would be better to always have the text first and then have the picture below that. right now, it's always picture first, and then text

I'm intrigued by your CI -- does the deploy preview automatically refresh? (I'm not seeing some of the changes that I'd expect to see based on the commits)

(for instance, the CNCF logo still looks like the original one instead of the colour one that Uche updated)

(for instance, the CNCF logo still looks like the original one instead of the colour one that Uche updated)

did you try disabling the cache?

I'll try that -- but some things did update 😅

I see the updates now.
I'm not sure about the AI generated images we're introducing. I don't think that they hold to the nautical theme that the project has historically had (I understand potentially wanting to move away from it, but if that's the case we're only partly moving away from it).

The metrics image doesn't speak to metrics (and I'm not sure i understand the cat being there)
The deployment checks doesn't really speak to what's happening (it reads more like checking a developer than checking a deployment)

The lighthouse theming and icons on kptn.sh are pretty strong, I'd be hesitant to move away from them.
Could we hold for a more thorough design review before merging this?

@nate-double-u I'd be glad to have better images! This is the best we could come up with as engineers 😅
If we get some design support and maybe therefore better graphics, I'm all for it!

I do like the looks of this!  Thanks so much!  I guess I don't really understand how the graphic for Metrics and Orchestrate relate to the theme but they are attractive and I, personally, like the cat ;-)
A couple verbage and content-flow questions for your consideration.  None of these are show-stoppers:

For the "Deployment Observability" block:

The second bullet is a verb whereas the first and third bullets are nouns.  Perhaps reword the second bullet to be "Troubleshooting assistance for failed deployments" but that might be too long.  I guess just "Troubleshooting failed deployments" works although it is less precise.
Shouldn't these items spell out "OpenTelemetry" rather than using "OTel"?  I googled "OTel" and it does eventually get one to the OpenTelemetry page but the first hits are to various hotel booking pages.  I'm not sure what the rules are for using the "OTel" string and the cognoscenti will no what we mean but it seems that our landing page should use the official name

When the user clicks on "Get Started", they go into the "Get started" section of the docs.  This makes some sense but it would be better if they read the "Core Concepts piece before they dig into the tutorials -- it kind of paints the big picture of what Keptn can do and all.  I see two options:

Make "Get Started" link to "Core Concepts".  Each subsection of the latter doc links to the "Get started" exercise for that component
Add a sentence like the following to the "Get started" landing page: "Before attempting these tutorials, we recommend that you read the "Core Concepts[link]" section
Yes, this is outside the scope of this PR.  I created https://github.com/keptn/lifecycle-toolkit/issues/3867 for it

Under "Learn from the Community", the "Discover More" button takes me to the Keptn youtube channel, which is still kind of a mess.  The three videos that I see without scrolling are "Migrating experience from Spinnaker to Keptn", the "Keptn New Contributor" meeting from Sept 11 2023 -- which is the one where Ondrej walks through the process of setting up a development environment so useful but perhaps should be re-titled or at least annotated, and a "Keptn in Google Summer of Code" from April 2022.

Yes, the mess that is the youtube channel is beyond the scope of this PR unless we want to curate the videos that get top billing

Should we have a link to the blogs?  Yes, they have a tab in the Documentation so I don't know...

  You can search for "WARNING" in the log to see them quickly

Hi @StackScribe
I'm actually working on a new design of the website. Will open another PR soon

@thisisobate shall we close this one in the meantime?

sure...feel free


Can't time-consuming operations be performed in the business initialization interface?

In the initialize() function, the loop sleep test always turns off after a few seconds. Is there any timeout setting for other components? Version 2.4 used.

Initialization cannot take too long, which will be killed by tarsnode.

初始化不能有太耗时的操作，这样会被tarsnode杀掉。


Organisation level workflow

Description
We should have the ability to set this up at an organisation level.
When we trigger a workflow we should go through every repository in the org (allow / disallow lists?) and run the standard workflow on each of those repositories.
Propose Solution
For this to work, we need to know all the repos that we want to review and perform the compliance check against.
Action Items:

[ ] Get all repository names
[ ] #20
[ ] allow / disallow lists?

[ ] Importable from file?

[optional] Alternative Solutions
A clear and concise description of any alternative solutions or features you've considered.

@leostolyarov Agree. After looking at some of the limitations for how many requests the Action would have to make this seems to me unlikely to work very well (especially for large orgs).
I much prefer your idea of just using a reusable workflow versus this issues original idea.


Feature/593 copy subscriptions context menu item

Feature: Add context menu item to copy subscriptions
This PR adds a new button to the context menu of a subscription and allows to duplicate the selected subscription.
Clicking the button prefills the regular "Create Subscription" form and creating a subscription using the new "Duplicate Subscription" form also copies all rules and actions associated to the original subscription to the target subscription.
Implements #593
Screenshots

:white_check_mark: Build ServiceBusExplorer 5.1.0-593-copy-subscriptions-context-menu-item.57.build.882 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/7fe704b002 by @jcoeltjen)

:white_check_mark: Build ServiceBusExplorer 5.1.0-593-copy-subscriptions-context-menu-item.56.build.883 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/344d602820 by @jcoeltjen)

:white_check_mark: Build ServiceBusExplorer 5.1.0-593-copy-subscriptions-context-menu-item.56.build.884 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/84755d064f by @jcoeltjen)

Great.
It would also be great @jcoeltjen, if you added the same functionality for queues and topics to get feature parity.

You are right. I will definitely look into that.
As the changes in the UI are quite specific for each case I will probably open two separate PRs for the changes.
The SB Explorer was tremendously useful for me in multiple cases already and I am looking forward to contribute more for this great project!


import error

fatal: bad object HEAD
Traceback (most recent call last):
File "testszfm.py", line 3, in
from pixell import enmap, utils, bunch, pointsrcs
File "/global/cscratch1/sd/kjc268/pixell/pixell/pointsrcs.py", line 29, in
from . import utils, enmap, srcsim, wcsutils
ImportError: cannot import name 'srcsim' from 'pixell' (/global/cscratch1/sd/kjc268/pixell/pixell/init.py)
srcsim is in pixell/cython while the other imports are in pixell/pixell

Hi @katiejchapman, what version of pixell are you using?
The version that works for me is 0.16.0
you can check this in python by:

import pixell
print (pixell.version)


Allow selection of location or buoy

Is your feature request related to a problem? Please describe.
A user should be able to select a location from a list.
Describe the solution you'd like
Put a select element and populate it with all locations for now (~80 locations)
This should be on the "home" page initially.
Describe alternatives you've considered
Search box is a currently implemented alternative, however it has some short comings. Eg., limited to searching for text in name column.
Additional context
Start with select, the consider other alternatives such as selectable markers from a map.

 That can be changed if i write it into admin panel.

I DID THIS!


Do USE_(DT_)CODE_PARTITION and FLASH_LOAD_OFFSET/SIZE need to be user-configurable?

See discussion near the end of https://github.com/zephyrproject-rtos/zephyr/issues/20673. It might be possible to improve https://github.com/zephyrproject-rtos/zephyr/pull/20740 some more if they weren't (though it already improves the situation, so no need to block it I think).
Recording it mostly so I don't forget it, and in case someone jumps in.
CC @novalisek

Having USE_DT_CODE_PARTITION user-configurable seems the sketchiest out of them.


Update GOT to resolve: https://github.com/sindresorhus/got/issues/1118

I am unable to use this library in some environments (even using all defaults) due to the aforementioned issue in GOT.

Thanks i ended up wanting to update everything

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


refactor(exthost): Add $deltaExtensions API

The vscode extension host exposes a deltaExtensions API - an API where we can load extensions on-demand (right now, we only load extensions on startup).
This is helpful for two use cases for us:

When installing an extension, we can activate it immediately, without needing the user to restart the extension host or editor
We could make our extension loading on startup asynchronous, and stream extensions to the deltaExtensions API as we load (right now, extension loading is totally synchronous on startup - not ideal for startup perf!)

This adds an API for calling into via our Exthost module, along with some simple test cases.

/azp run


ESP32 I2C issues

Hello,
I am trying to use your library on my ESP32 and I'm having issues when compiling it properly.
See below for my errors.  I chose the ESP32 Dev Module when compiling.  This is the basic serial example butI get the same errors when using it in my custom sketch too.
/Users/gregoryhulette/Documents/GitHub/Arduino-Code/libraries/HumanCyborgRelationsAPI-main/src/hcr.cpp: In member function 'void HCRVocalizer::begin(uint16_t)': /Users/gregoryhulette/Documents/GitHub/Arduino-Code/libraries/HumanCyborgRelationsAPI-main/src/hcr.cpp:111:23: error: 'class TwoWire' has no member named 'setWireTimeout'; did you mean 'setTimeout'?                  _i2c->setWireTimeout(3000 /* us */, true /* reset_on_timeout */);                        ^~~~~~~~~~~~~~                        setTimeout /Users/gregoryhulette/Documents/GitHub/Arduino-Code/libraries/HumanCyborgRelationsAPI-main/src/hcr.cpp: In member function 'void HCRVocalizer::transmit(String, bool)': /Users/gregoryhulette/Documents/GitHub/Arduino-Code/libraries/HumanCyborgRelationsAPI-main/src/hcr.cpp:187:9: error: jump to case label [-fpermissive]          default: break;          ^~~~~~~ /Users/gregoryhulette/Documents/GitHub/Arduino-Code/libraries/HumanCyborgRelationsAPI-main/src/hcr.cpp:168:17: note:   crosses initialization of 'int i2cStatus'              int i2cStatus = 0;                  ^~~~~~~~~ /Users/gregoryhulette/Documents/GitHub/Arduino-Code/libraries/HumanCyborgRelationsAPI-main/src/hcr.cpp: In member function 'void HCRVocalizer::receive()': /Users/gregoryhulette/Documents/GitHub/Arduino-Code/libraries/HumanCyborgRelationsAPI-main/src/hcr.cpp:229:9: error: jump to case label [-fpermissive]          default: break;          ^~~~~~~ /Users/gregoryhulette/Documents/GitHub/Arduino-Code/libraries/HumanCyborgRelationsAPI-main/src/hcr.cpp:210:17: note:   crosses initialization of 'int bytes'              int bytes = _i2c->requestFrom((int)_i2caddr,(int)8);
Thanks

I have the same problem with ESP32 board and I2C. Can't compile. Errors like greghulette wrote.

Sourcing a ESP32 to test, will be able to correct soon


Generate and review validation reports: Nuxeo UCB ESL (repository #7)

Step 2: Generate reports

[x] Generate validation reports (run validate_by_mapper DAG): https://7a8067cb-3b99-477e-a883-7e311175a9b4.c3.us-west-2.airflow.amazonaws.com/dags/validate_by_mapper_type/grid?dag_run_id=manual__2024-01-19T22%3A08%3A46%2B00%3A00
[x] Add a comment (in this issue) to the output URLs
[x] Download the validation reports locally; prepend the collection registry number to the filename
[x] Upload validation reports to the Rikolti Data Validation folder

Step 3: Review reports

[x] Review sample collections validation reports; leave notes in the spreadsheet
[x] Quickly (and not very deeply) scan through other validation reports (to identify if any new metadata fields surface)
[x] Synthesize notes at the mapper level
[x] Share an update (and surface any anomalies) during a weekly check-in (Mondays at 3:00pm)

Step 4: Have data validation issues? Address & resolve validation issues (synthesized mapper-level):

[ ] Create a new issue (“Validation results: Rikolti mapper name“) and report/request fixes & investigations. Add to current sprint.
[ ] Add a comment to link to the new “validation results” issue.
[ ] Add a “blocked/paused” label to this issue. Include a status note in this issue title (e,g., HOLD for fixes; HOLD for code review; etc.)
[ ] Move this issue back to the “To Do” pile.
[ ] Reset tasks listed in steps 2-3 above; re-generate & re-review (repeat steps 2-3 above, as needed.)

Step 5: Have acceptable validation reports?

[x] Move the valid (accepted) reports to the “VALID” folder in google drive.

[2024-01-19, 22:09:24 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/27312/vernacular_metadata_2024-01-19T22:09:00/mapped_metadata_2024-01-19T22:09:14/validation_2024-01-19T22:09:23.csv
[2024-01-19, 22:09:24 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#27312/vernacular_metadata_2024-01-19T22:09:00/mapped_metadata_2024-01-19T22:09:14/data/
[2024-01-19, 22:09:24 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/27807/vernacular_metadata_2024-01-19T22:09:04/mapped_metadata_2024-01-19T22:09:16/validation_2024-01-19T22:09:23.csv
[2024-01-19, 22:09:24 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#27807/vernacular_metadata_2024-01-19T22:09:04/mapped_metadata_2024-01-19T22:09:16/data/
[2024-01-19, 22:09:24 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/28042/vernacular_metadata_2024-01-19T22:09:04/mapped_metadata_2024-01-19T22:09:16/validation_2024-01-19T22:09:24.csv
[2024-01-19, 22:09:24 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#28042/vernacular_metadata_2024-01-19T22:09:04/mapped_metadata_2024-01-19T22:09:16/data/

New reports ready for re-review:
[2024-01-31, 00:59:19 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/27312/vernacular_metadata_2024-01-31T00:58:56/mapped_metadata_2024-01-31T00:59:09/validation_2024-01-31T00:59:18.csv
[2024-01-31, 00:59:19 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#27312/vernacular_metadata_2024-01-31T00:58:56/mapped_metadata_2024-01-31T00:59:09/data/
[2024-01-31, 00:59:19 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/27807/vernacular_metadata_2024-01-31T00:58:59/mapped_metadata_2024-01-31T00:59:11/validation_2024-01-31T00:59:19.csv
[2024-01-31, 00:59:19 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#27807/vernacular_metadata_2024-01-31T00:58:59/mapped_metadata_2024-01-31T00:59:11/data/
[2024-01-31, 00:59:19 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/28042/vernacular_metadata_2024-01-31T00:58:59/mapped_metadata_2024-01-31T00:59:11/validation_2024-01-31T00:59:19.csv
[2024-01-31, 00:59:19 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#28042/vernacular_metadata_2024-01-31T00:58:59/mapped_metadata_2024-01-31T00:59:11/data/

Accepted on 2024-02-05!


Add the for..of statement in *-curly-braces rules

Add the for..of statement in disallow-curly-braces and require-curly-braces rules.

:+1: Thanks for contributing! - And if you want can you change the commit message to be like disallowCurlyBraces: support for of statement?

The commit messages are changed.

Landed, thank you


Updated Change Log for 0.2.5

closes #53

I compiled a rough list from the commit logs, so hopefully I didn't miss too much. Let me know of any changes that need to be made.
On a related note, is there any reason why the changelog is manually updated in the README instead of either in tag comments (like three.js) or in a Changelog.md? I think the former is a better way to go and would hopefully make it more likely to actually tag the releases (nothing before 0.2.5 is tagged AFAICT).
Also, before I open an issue for it, is there any chance in getting a new release any time soon? I'd very much like to see #162 land in a new release so I can lock my app to it. :smile: If there's no major reason why a release hasn't been cut, I'll open an issue for it.

So is this changelog for the respective tag https://github.com/threeDart/three.dart/releases/tag/v0.2.5%2B1 ?
Looks good to me, but as what version do we want to release the current master? 0.3.0 maybe?

Yes, I just looked at the diff between that tag and 483ca5d1230528061be129c59d1eb4e5686739b0
(which was the first commit after 0.2.4 was publish to pub.  I assume that was the tag that was published to pub. I didn't bother separating 0.2.5 and 0.2.5+1 since they were so close together.

but as what version do we want to release the current master? 0.3.0 maybe?

I think 0.3.0 is probably the best. Looking through the commit log, most of the changes were bug fixes, but there were quite a few new features and some refactoring.
I've been working on a lot of documentation, so it might be nice to wait for that (mostly copy-paste from three.js documentation, but I also added lots of type annotations). I should have a pull request ready soon.

I think we should hold a bit longer and run dartfmt before tagging and publishing 0.3.0

Any update on this? If I've missed something, please let me know and I'll fix it.
I feel this needs to get done before the next release is even considered.


What does it mean whether LR is required in the configuration file?

How can this code be changed to conditional generation? I changed whether LR is required in the train to LRHR, and the result is an error, can this LRHR only be used in val? Please give me some pointers

Do you fix this bug?

I've noticed this issue too. Have you managed to resolve it?

Has anyone solved this problem？

你修复了这个错误吗？

Do you have any opinion on this issue？

我也注意到了这个问题。你解决了吗？

Do you have any opinion on this issue？


Standalone contacts/address book like other email clients

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
6.201
Problem you are trying to solve
I'd always had my email addresses stored with my email account/client, and my phone numbers stored on my phone. But with K-9 mail, I'll have to merge these, which I don't really want to do. Currently I have to memorise emails or copy and paste then. Other people might also want this for privacy reasons (don't want to give the app the contacts permission).
Old related issue: https://github.com/thundernest/k-9/issues/2560
Suggested solution
Having an option to use a standalone address book like gmail, outlook, thunderbird etc would be great.
Screenshots / Drawings / Technical details
No response

Android works differently than desktop systems in regard to address books. No matter how it is done, an integrated address book will never be as powerful as the solution that Android already ships with. I would be disappointed if K-9 started shipping its own solution because it would be wasted man-power to develop something less flexible.
Your use case can be solved by adding a second address book to Android that you store the email addresses in, and which you hide in the contacts app.

How would I do that? I don't see an option in the contacts app to create a second address book/set of contacts.

How would I do that? I don't see an option in the contacts app to create a second address book/set of contacts.

Android allows adding multiple "contact providers". You could try using a "provider" that doesn't sync to the cloud but only stores contacts locally, e.g. https://play.google.com/store/apps/details?id=com.fjsoft.mylocalaccount

How would I do that? I don't see an option in the contacts app to create a second address book/set of contacts.

Android allows adding multiple "contact providers". You could try using a "provider" that doesn't sync to the cloud but only stores contacts locally, e.g. https://play.google.com/store/apps/details?id=com.fjsoft.mylocalaccount

I've installed that, added its account to the account list in settings and added a test entry. But there doesn't appear to be a way to hide the entries from the contact app, and the contact does not show up in K-9 Mail.

Depends on what contacts app you are using. The default one from Google has it in the contacts app's settings, under "Contacts to display".

This is what K9 needs https://github.com/thundernest/k-9/issues/6400

@cketti This solution does not appear to work. Please could this issue be reopened?

Yes, please reopen!
I want to use k9 because I don't want to use the Android or Google default solutions, so I hope still for quite a long time for a standalone address book in k9.
Just a very simple one would do. I don't need an address book with relations to all my other apps, but I am in urgent need for k9 saving my contact e mail addresses.

Yes, please reopen!
I want to use k9 because I don't want to use the Android or Google default solutions, so I hope still for quite a long time for a standalone address book in k9. Just a very simple one would do. I don't need an address book with relations to all my other apps, but I am in urgent need for k9 saving my contact e mail addresses.

For now I'm using a workaround of storing emails in a text file and copying and pasting them. But as you say, it'd be great to have an option for this (like most email clients have, at least on PC).


Worker script fetching from /links giving timeout

I am done with all the setup (HTML and API) but stuck with a small issue. I have /links and the HTML page in the same worker script. After writing logic for /links I published the worker on my dev setup. Now for HTML if I try to fetch from /links from my published project I am getting timeout error.
But if I preview the same page from the Cloudflare dashboard under "Quick Edit" option for my worker I am able to see the page since the fetch works that time.
Not sure what to do with this?

First I would say restart your local server instance from your wrangler dev command. If that doesnt work, can you print to the console in your /links logic?

So /links is working both on local machine and on .dev. It's just that while rendering the HTML, I do a fetch to my /links end point (https://engineering-assignment.adarsh-trivedi100.workers.dev/links) and only in that case I am getting timeout. That too the timeout is not happening when I preview my HTML page link (https://engineering-assignment.adarsh-trivedi100.workers.dev) from the preview section in my worker on cloudflare dashboard.

Do we have to use a GET request to get the links data? There is conflicting information in the instructions.
The picture below says links array should be available as variable for both endpoints.

This piece in the instructions says to get it as JSON from the /links endpoint.

Which one are we supposed to do?

Do we have to use a GET request to get the links data? There is conflicting information in the instructions.
The picture below says links array should be available as variable for both endpoints.

This piece in the instructions says to get it as JSON from the /links endpoint.

Which one are we supposed to do?

I mean I assumed we need to do it the way specified in later part. Can we do it either ways?

@jptboy Ive assumed the variable instructions. It doesnt make sense for the worker to call itself for information it should already have.

@perryBunn I assumed the same. Calling itself would be a weird thing to do.
Also the "variable" instructions are after the "get links from json" instructions.

Try calling the function directly rather than a separate request to /links.
Joaquin
On Tue, Oct 20, 2020 at 1:02 PM 'Adarsh Trivedi' via
cloudflare-assignment-application <
<EMAIL_ADDRESS>wrote:

I am done with all the setup (HTML and API) but stuck with a small issue.
I have /links and the HTML page in the same worker script. After writing
logic for /links I published the worker on my dev setup. Now for HTML if I
try to fetch from /links from my published project I am getting timeout
error.
But if I preview the same page from the Cloudflare dashboard under "Quick
Edit" option for my worker I am able to see the page since the fetch works
that time.
Not sure what to do with this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cloudflare-hiring/cloudflare-2020-general-engineering-assignment/issues/14,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARBVIAPAFCRBWKLNXEMDYQLSLXGCNANCNFSM4SYWBOFA
.
--
You received this message because you are subscribed to the Google Groups
"cloudflare-assignment-application" group.
To unsubscribe from this group and stop receiving emails from it, send an
email to<EMAIL_ADDRESS>To view this discussion on the web visit
https://groups.google.com/a/cloudflare.com/d/msgid/cloudflare-assignment-application/cloudflare-hiring/cloudflare-2020-general-engineering-assignment/issues/14%40github.com
https://groups.google.com/a/cloudflare.com/d/msgid/cloudflare-assignment-application/cloudflare-hiring/cloudflare-2020-general-engineering-assignment/issues/14%40github.com?utm_medium=email&utm_source=footer
.
For more options, visit
https://groups.google.com/a/cloudflare.com/d/optout.

--
Joaquin Madruga | Engineering Director
<EMAIL_ADDRESS>1 888 99 FLARE  |  www.cloudflare.com

Hi,
I'm trying to test my project by running 'wrangler dev' in the command line, but I am receiving an error saying that my account is not entitled to use this method, and that I need to enable workers bundled for my account. I've been trying to troubleshoot this by using the documentation, but have been unsuccessful.
Brendan Carey

This is likely because you used a Workers Sites template for your project
which depends on Workers KV.  Workers KV is currently a paid product.
Joaquin
On Fri, Oct 23, 2020 at 3:22 PM 'brendancarey19' via
cloudflare-assignment-application <
<EMAIL_ADDRESS>wrote:

Hi,
I'm trying to test my project by running 'wrangler dev' in the command
line, but I am receiving an error saying that my account is not entitled to
use this method, and that I need to enable workers bundled for my account.
I've been trying to troubleshoot this by using the documentation, but have
been unsuccessful.
Brendan Carey
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cloudflare-hiring/cloudflare-2020-general-engineering-assignment/issues/14#issuecomment-715571676,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARBVIANM3NUN54FJZZ7U3ZTSMHQWNANCNFSM4SYWBOFA
.
--
You received this message because you are subscribed to the Google Groups
"cloudflare-assignment-application" group.
To unsubscribe from this group and stop receiving emails from it, send an
email to<EMAIL_ADDRESS>To view this discussion on the web visit
https://groups.google.com/a/cloudflare.com/d/msgid/cloudflare-assignment-application/cloudflare-hiring/cloudflare-2020-general-engineering-assignment/issues/14/715571676%40github.com
https://groups.google.com/a/cloudflare.com/d/msgid/cloudflare-assignment-application/cloudflare-hiring/cloudflare-2020-general-engineering-assignment/issues/14/715571676%40github.com?utm_medium=email&utm_source=footer
.
For more options, visit
https://groups.google.com/a/cloudflare.com/d/optout.

--
Joaquin Madruga | Engineering Director
<EMAIL_ADDRESS>1 888 99 FLARE  |  www.cloudflare.com


samlssoTokenId cookie is not cleared upon logout when Front-Channel Logout Method is selected

Describe the issue:
samlssoTokenId cookie is not cleared upon logout when Front-Channel Logout Method is selected. This results in the following behavior.
Consider a scenario where there are two applications, App A and App B. Login to App A and then log out from it. Thereafter login to App B and log out from it. Since the samlssoTokenId cookie is not cleared during the previous log out from App A, the Identity server tries to logout the App A first(however the user has already been logged out from App A)
How to reproduce:

Set up a SAML service provider that supports the front channel logout[1].
Login to the service provider and then log out.
Notice the samlssoTokenId cookie is not cleared upon log out.

Expected behavior:
The samlssoTokenId cookie should be cleared in the above scenario upon logout.
Environment information :

Product Version: [IS-5.3.0,IS-5.10.0]

[1]https://dewni-matheesha.medium.com/saml-2-0-front-channel-logout-with-wso2-identity-server-8a72d1075cf5

This is done for the 5.10 milestones as well. Refer here. @AnuradhaSK you can close this issue.


Component: Select (not synced with design)

Description of the bug

Tab-focus (purple outline) should only show when using keyboard navigation, not when you activate the select-box.

When resizing window (storybook) the component seems to be a fix width and does not scale.

There should be two sizes: Medium with 18px, and Small with 16px font size both for label, description and options. Seems like the options is larger than the selected text.

The X on the chip is not centered

Should be named "Single select" and "Multi select" instead of "Flervalgsmeny" and E"nkel"

See Figma sketches for correct design:
https://www.figma.com/file/vpM9dqqQPHqU6ogfKp5tlr/DDS---Core-Components?node-id=7577%3A43639&t=xlYDTn7VXmpckqPy-1
Steps To Reproduce
Go to https://digdir.github.io/designsystem/?path=/story/kjernekomponenter-select-flervalgsmeny--normal to reproduce the behavior.
Additional Information
No response

Just some remarks on points 2 and 5:
2 (resizing): This is only an issue in Storybook. The actual component defaults to the width of its container. To make the example in Storybook look more realistic, there is an invisible container with a fixed width around the select box.
5 (naming): We have chosen to use Norwegian as the default language in Storybook, except from components, which are referred to by their name in the code. Single select and multiple select are just variants of the same component, Select, hence the choice of menu item names.

Point 1 will probably be solved by #129.

Not relevant anymore, since we have updated design in Figma. New issue will be created with updated design.


Unable to fetch content files

Greetings, first wanna give props for providing a great project + case study on WebXR 😤👍
Been having difficult fetching the content files. At first I thought something was broken because the blends didn't open anything and the glTF files missing textures and PNGs didn't open in image viewer, then realized I needed git-lfs (should be noted in the README). I've tried installing git-lfs, doing git lfs pull / fetch / clone / checkout, and get errors like this:

Am I doing something wrong? I'm using Windows + WSL2 btw.
Also have a recommendation: for better artist accessibility IMO the blend files could be shared on Sketchfab, there's no easy way to use git-lfs (GitHub desktop doesn't provide any interface) and command line is usually a show stopper for artists. Also many tools / engines / platforms have importers that make it seamless to share and use in our workflows. Plus it can help with discoverability!

Thank you for reporting this issue, I verified it just now and there is indeed a problem with files pushed to Github. I will open a task internally to get this resolved as soon as possible.
The accessibility aspect is also a good point. Though if the files were pushed correctly, regardless of whether lfs is enabled for this repo, people should be able to get everything by clicking Code -> Download Zip on the repo front page to its content.

Unfortunately downloading the repo zip directly still only gives you the LFS pointers, not the original files.

This issue should now be resolved, thank you @madjin again for flagging this.