BlueGreen flag

this is for https://github.com/replicate/cluster/pull/1198
I almost rather want to make lookupDefault public, this is still a little awkward

Yeah, this is a little awkward. Are we going to use a string key (eg values like "cog-0.9b17") or a boolean key, where true means blue (they rhyme!) and false means green?
If we're using a boolean, would it be cleaner a bit like this?
func BlueGreen[T any](context *ldcontext.Context, name string, blueValue T, greenValue T, defaultToBlue bool) T {
	if lookupDefault(context, name, defaultToBlue) {
		return blueValue
	} else {
 		return greenValue
	}
}

If we're using a string, maybe we don't need to do the blue green dance at all? Instead we have a default value in the code, but we normally just take whatever string LaunchDarkly gives us. Then maybe we just expose a more raw LookupString method from flags:
func LookupString(context *ldcontext.Context, name string, defaultVal string) string {
	log := logger.Sugar()

	if currentClient == nil {
		return defaultVal
	}
	if context == nil {
		log.Warnw("flags package was passed a nil context: returning default value", "flag", name, "default_value", defaultVal)
		return defaultVal
	}
	// StringVariation and friends only return an error in the event that flags
	// are not available (e.g. if LaunchDarkly is having an outage or we've
	// misconfigured the client).
	result, err := currentClient.StringVariation(name, *context, defaultVal)
	if err != nil {
		log.Warnf("Failed to fetch value for flag '%s' (returning default '%s' to caller): %v", name, defaultVal, err)
	}
	return result
}

per @nickstenning storing non-bools in LD is verboten
func BlueGreen[T any](context *ldcontext.Context, name string, blueValue T, greenValue T, defaultValue T) T would be fine
do you think you would be willing to pick this up? I'm not terribly comfortable with this codebase and just want to get cog upgrades unblocked


All write-related filesystem errors are being reported as "permission denied"

Describe the bug
All write-related filesystem errors are being reported as "permission denied"
https://github.com/unvt/charites/blob/88c921556e856f0eea17de98430e091e4a13fdd4/src/lib/yaml-writer.ts#L33-L37
To Reproduce
Run convert with a destination of a volume that is full or read-only
Expected behavior
A detailed error describing that the volume is full
Charites Version:

Charites version: 0.X.X

Desktop (please complete the following information):

OS: [e.g. Windows]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
I think that the underlying error should be exposed instead of overwriting it. This has lead to confusing error reports: #56 , where the error reports "permission denied" when it should ENOENT: no such file or directory (that bug was because the parent directory wasn't being created, which throws the no such file or directory error)

Dear @keichan34  and @miya0001 ,
Thank you very much for improving charites!
If it is possible, can you also update npm package so that we can use the charites with improvement by the pull request #129?
https://github.com/unvt/charites/pull/129
I can use it by cloning the repository, but it would be easy for users to install via npm.
Thank you very much for your kind attention!
Best regards, Taro Ubukawa

@ubukawa Sorry for the late release! I just merged a couple outstanding pull requests and created a new release 0.3.0 :)

Dear @keichan34 ,
Thank you very very much for your prompt response!
I really appreciate your continued contribution.


Running YCSB Workload

Whenever we are setting the operation count property, is it for a particular thread or for all the threads (as defined using the threadcount property)?
If it is collectively used for all the threads, how can I specify the number of operations each thread should perform?

it's across all client threads. the operations are always uniformly distributed.

Thank you for your reply @busbey !


Zap receipts generated as if sending to self / incorrect p tag use

client initiating action
Damus + alby
client receiving or observing action (or lack thereof)
Damus
timestamp
what happens
A notification is rendered when sending zaps to someone else, as if zapping oneself.
suggestion
Modify p tag application.

@bumi


Documentation issue - DOCX Templates in AssemblyLine

Describe the bug
Difficulty customizing templates in AssemblyLine.
To Reproduce
---
include:
  - docassemble.AssemblyLine:al_package.yml
---
mandatory: True
code: |
  tab_intro
  thumbnails_defaults
  download_list_defaults
  end_tab
---
objects:
  - exhibits_bundle_defaults_1: ALDocumentBundle.using(elements=[exhibit_doc_defaults_1], filename="exhibits_bundle_defaults", title="Exhibits with defaults")
  - exhibit_doc_defaults_1: ALExhibitDocument.using(title="Exhibits doc defaults", filename="exhibits_doc_defaults", include_exhibit_cover_pages="True" )
---
object: ALExhibit
attachment:
  variable name: x.cover_page
  docx template file: exhibit_cover_sheet_APS.docx
  field code:
    exhibit_label: raw(x.label)
---
object: ALExhibitDocument
attachment:
  variable name: x.table_of_contents
  docx template file: exhibit_table_of_contents_APS.docx
  field code:
    exhibits: raw(x.exhibits)
    include_exhibit_cover_pages: raw(x.include_exhibit_cover_pages)
---
field: tab_intro
mandatory: True
question: Tabbed document generator
subquestion: |
  This tabbed document generator creates a single PDF out of multiple documents. It will automatically insert a tab for each document and include a tab sheet in the generated PDF. It will also provide a table of contents for the PDF. [BR][BR]
  *Be careful to review the output PDF. This generator will only work as well as you input information. If any inconsistencies or misdescriptions exist, you must catch them and correct them.*
---
id: thumbnails_defaults
continue button field: thumbnails_defaults
question: Please review the compiled tabs.
subquestion: |
  You will be able to download these documents in their final form on the next screen.[BR][BR]
  ${ exhibits_bundle_defaults_1.as_pdf() }

  ${ exhibits_bundle_defaults_1.as_zip() }
  [BR][BR]If changes are required, select the "undo" button.
---
id: download_list_defaults
continue button field: download_list_defaults
question: Download the completed document.
subquestion: |

  ${ exhibits_bundle_defaults_1.download_list_html() }
---
id: end_tab
event: end_tab
question: You have successfully compiled your documents into a single tabbed document.
buttons:
  - Restart: restart
---

Expected behavior
The templates specified in the object blocks for ALExhibit and ALExhibitDocument should be used in stead of the default templates.
Version Information (should be in your.docassemble.server/updatepackage:

Docassemble Version: 1.4.111
AssemblyLine Version: 2.28.1

Additional context
This may be more of a documentation issue for those, like me, who do not have the technical skill to delve too deeply into Python. The few lines of text relating to changing template files in the documentation do not have enough detail for me to know which actions to take.
I have uploaded my custom templates to Docassemble's template folder and to the template folder created in docassemble/AssemblyLine/data/templates. Neither upload is used.
As I say, I am likely missing some code that points to the correct template. I cannot, however, discern enough from the below documentation to customize the template.

Customizing Exhibits
You can add your own template for the table of contents by customizing your_document.cover_page. Use the stock table of contents template as a model. It has some fancy math to figure out what page number to start the exhibits at, based on whether it thinks the table of contents itself will spill onto a second page.
You can add your own template for the individual cover pages for each document, too. Use the stock cover page template as a model.

Hi apstrom, you've changed generic object in our code to object in your code which is what caused your problem. object isn't a Docassemble keyword.
You don't need to use generic object here--you can directly specify the attribute. x has a special meaning in Docassemble for generic object questions only. You'd substitute your actual document's name for the x.
---
attachment:
  variable name: exhibit_doc_defaults_1.table_of_contents
  docx template file: exhibit_table_of_contents_APS.docx
  field code:
    exhibits: raw(exhibit_doc_defaults_1.exhibits)
    include_exhibit_cover_pages: raw(exhibit_doc_defaults_1.include_exhibit_cover_pages)

Thanks!
Is this solution equally applicable to the exhibit cover page template?

Still having an issue with the cover page template. DocAssemble is not pulling the correct template (from the DA templates folder, not AssemblyLine's folder).
I am trying to swap the word "exhibit" in the cover page for "TAB", which allows ALExhibitDocument to be used for things like motion records or compendia. Whatever I do, the output returns the stock AssemblyLine cover page template.
This issue has been driving me nuts. Any help would be much appreciated.
My code:
---
objects:
  - exhibits_bundle_defaults_1: ALDocumentBundle.using(elements=[exhibit_doc_defaults_1], filename="exhibits_bundle_defaults", title="Exhibits with defaults")
  - exhibit_doc_defaults_1: ALExhibitDocument.using(title="Exhibits doc defaults", filename="exhibits_doc_defaults", include_exhibit_cover_pages=True, include_table_of_contents=True,)
---
attachment:
  variable name: exhibit_doc_defaults_1.cover_page
  docx template file: exhibit_cover_sheet_APS.docx
  field code:
    exhibit_label: raw(exhibit_doc_defaults_1.label)
---
attachment:
  variable name: exhibit_doc_defaults_1.table_of_contents
  docx template file: exhibit_table_of_contents_APS.docx
  field code:
    exhibits: raw(exhibit_doc_defaults_1.exhibits)
    include_exhibit_cover_pages: raw(exhibit_doc_defaults_1.include_exhibit_cover_pages)
---


Remove byte[] support

Use CC 1.76+'s byte ⇌ char conversion instead. There should be a switch to use this globally on 1.75.

I'm not going to remove byte[] support: it feels better to rely on not converting at all. I am going to phase out IBinaryHandler though.


Html file lost in the book structure

There is a moments.html in the shearbook directory.
Shall we remove it ?

@aboucaud maybe I can just include that fix in #4

please do

File removed in #13


Wiki correction for ADO Pipeline

Hey @kamil-mrzyglod :) Hope you are good!
The Wiki needs amending I think, as the later versions do not contain 'ace' in the name of the zip file.
https://github.com/TheCloudTheory/arm-estimator/wiki/Installation
  - task: Bash@3
    inputs:
      targetType: 'inline'
      script: |
        wget https://github.com/TheCloudTheory/arm-estimator/releases/download/1.5/linux-x64.zip
        unzip linux-x64.zip
        chmod +x ./azure-cost-estimator
      workingDirectory: $(System.DefaultWorkingDirectory)

Fixed, thanks!


Feature: routed modals + parameters in modal routes

Loving this package so far. I generally prefer to have modals routed though rather than state based in order to have shareable links as a default, rather than state-based (which are only good for global routes).
I have managed to do this myself with custom routes file:
E.g., I have a route in pages/companies/+edit.[id].tsx
And then in routes file I do this to render modals for routes, following the internal matching engine of react-router-dom:
const modalsArray = Object.entries(modalRoutes).map(([path, Component]) => ({
  path: path
    .replace(...patterns.route)
    .replace(...patterns.splat)
    .replace(...patterns.param)
    .replace(/(^|\/)\-\:(:?[\w-]+)(\/|$)/, '$1$2?'),
  element: <Component />,
}))

export const Modals = () => {
  const location = useLocation()
  const matches = matchRoutes(modalsArray, location)
  return renderMatches(matches)
}

Would be great to have this with type-safety out of the box for linkable routes with params.

That's interesting! Could you share that via StackBlitz for example? I'll be happy to have a look into that.
I was also thinking recently to store current modal as a search params instead of route state to have it shareable.

I can set on up later. But after more exploration, it seems like the best way to add routed modals would be to add them as children to the parent, and rendering through an <Outlet /> in the parent. Then you can also easily close them by using navigate("..") to route back to the parent.
Check out this example from the official repo:
That would also support cases for routed tabs, etc. I don't think there's a convention to add nested routes right now, is there? Maybe something like index+modal.tsx?

After more testing, seems as if nesting works with, e.g. this structure:

companies (directory)

index.tsx
[id].tsx
[id] (directory)

tab1.tsx
tab2.tsx

The only problem is what if I want the index page to also have children components? That doesn't seem to be possible. The only option is to define company.tsx in root pages folder, but that makes everything in the company directory a descendant of that.
I think a good option would be to treat an index directory as a source for children for the index route. Right now if you would create an index directory in  e.g. companies, then it would create an erroneous route group with path "/", and that's not valid:
react-dom.development.js:26923 Uncaught Error: Absolute route path "/" nested under path "/companies" is not valid. An absolute child route path must start with the combined path of all its parent routes.

I think this example https://github.com/oedotme/generouted/tree/main/examples/react-location/modals might be using a similar approach to what you're referring to.
Basically to have an outlet and a child modal component, which gets mounted on top of it's parent.
If I recall correctly it had some limitations and it's wouldn't be global by default. It worked tho while I was testing that, and I guess it can be used right now with nested layouts.
Currently the global modals work well for me, but I'm interested to explore ways to improve it.

I am actually trying to do this right now. Is this something you would be interested in implementing? @oedotme
I am trying to do the following:
Kanban board at project/123
click on a card on the board and then a draw appears with the card information at project/123/card/456 however it looks like something like this solution would allow me to do that where as I am not sure at the moment the best way to do this with genrouted?

@archiekd modal routes can be achieved with route layouts + certain structure + just links and can be done with or without generouted. It's different than the existing global modals in generouted though, however you can combine both methods in the same application for different types of modals.
I added an example for react-router with route modals. In this example, /gallery is a layout route at pages/gallery.tsx with an <Outlet /> component. The modal route /gallery/:id at /pages/gallery/[id].tsx, is basically a "modal" or a "drawer" component that covers up the parent layout.
Here's the online example on StackBlitz

Hope that helps!

I managed to get it working mostly on my end without major modifications to generouted, but there's one use case still not properly supported. There's no way to add child routes to the index page.
Say you have routes for a tabbed view:

/pages/gallery.tsx (parent route, holding common UI for all tabs)
/pages/gallery/index.tsx (default tab)
/pages/gallery/tab1.tsx
/pages/gallery/tab2.tsx

Say I want to add a modal to the gallery page/tab and render the tab still in the background. For other tabs it's easy – add a folder [tab1] and e.g. file edit.tsx within there (+outlet in the tab).  There's no intuitive way to achieve the same for the index page though.
gallery/index.tsx is interpreted as an index: true path by default which cannot have any children. Adding a folder called index creates an erroneous path "/" which cannot exist at all as a nested path. I have monkeypatched it on my end with custom routes, by merging routes with erroneous path: "/" as children of index: true route on the same level.

All of your suggestions seem very delicate to me. I don't really understand if we are all trying to achieve the same goal, but I would rather explain my point of view on the current flows of the modal system.
In the context of my application, modals are sometimes shared between routes: I'd like to have a modal that deletes an item based on a given ID. To achieve this, I can do the following for a route: (dashboard)/items/[id]/+delete.tsx. This would work to delete my item in that modal, based on the id parameter in the URL, but if I'm in (dashboard)/items/index.tsx there's no way for the modal to get the id parameter.
In fact, your tricky way would work by setting a route page that is actually a modal (using the <Outlet />'). But isn't that confusing, since _layout.tsx' usually uses it? See what I mean? A route shouldn't act as a layout and it shouldn't act as a modal.
Solution :
A solution, but probably costly, would be to have some sort of context for each modal, so we can set them parameters without depending on the route URL.

So I found a strange way to deal with modal parameters on arbitrary routes by using history state :
(dashboard)/products/[id]/+delete.tsx
function DeleteProductModal() {
  let { id } = useParams("/products/:id");
  if (!id) id = history.state.usr.product.id;

(...)
}

(dashboard)/products/_components/ProductsTable.tsx
(...)
              <DropdownMenuItem
                className="hover:!bg-destructive hover:!text-destructive-foreground"
                onClick={() => modals.open("/products/[id]/delete", { state: { product: { id: product.id } } })}
              >
                <TrashIcon className="mr-2 h-4 w-4" />
                Delete
              </DropdownMenuItem>
(...)```

Well, what you're trying to achieve is a confirm box rather than a modal. And there's no reason to put it in a route to begin with – it's not like you want or need a shareable link for a confirm dialog (imagine a situation where deletion fails for some reason, if all else fails you'd rather want the user to get rid of it with a hard refresh rather than meeting the same bug over again). I think you're much better of just having a reusable confirm dialog component that defines the the ID on the trigger. You'll find it much easier to implement and don't have to have any hacky workarounds with IDs.
To that tune, I also personally don't think that global modals that don't have a URL should be handled by a routing library, as they may need additional state and if the state doesn't live in the URL, they become finicky. In that case you're also better off having reusable modal component for global modals that are tied to their triggers rather than awkwardly living in an additional routing tree.
Radix has good implementations for both that help you appreciate the paradigm:
https://www.radix-ui.com/primitives/docs/components/alert-dialog
https://www.radix-ui.com/primitives/docs/components/dialog
I would love to avoid having to add individual outlets for routed dialogs (albeit there's nothing particularly wrong with that approach, as you can have nested Outlets for many things, such as tabs, and other content(1) – it's just the additional boilerplate that bothers me, as modals are often placed in a portal anyways), but I'm yet to come to a better alternative which enables to properly store the state in and retrieve the state from the URL (and not history.state, which is not persistent, so I try to avoid that), and not having to create an alternative navigation logic on top of useNavigate.
--
1 - https://remix.run/ website demo illustrates a good example of outlet based composition for inspiration.


Bump LLVM to d13da154a7c7eff77df8686b2de1cfdfa7cc7029

Jenkins Linux amd64 Build #12875 [push] refactor: bump llvm (#25... started at 00:25

Jenkins Linux s390x Build #12899 [push] refactor: bump llvm (#25... started at 01:25

Jenkins Linux ppc64le Build #11892 [push] refactor: bump llvm (#25... started at 01:32

Jenkins Linux amd64 Build #12875 [push] refactor: bump llvm (#25... passed after 1 hr 57 min

Jenkins Linux s390x Build #12899 [push] refactor: bump llvm (#25... passed after 1 hr 59 min

Jenkins Linux ppc64le Build #11892 [push] refactor: bump llvm (#25... passed after 2 hr 56 min

@maxbartel big thanks for your help on this one, much appreciated as always.


CLUSTER_AUTHORIZATION_FAILED when producing.

Using franz-go 0.6.9 with Go 1.16
I create client like
	client, err := kgo.NewClient(
		kgo.SeedBrokers(fmt.Sprintf("%s:%d", address, port)),
		kgo.SASL(scram.Auth{User: username, Pass: secret}.AsSha256Mechanism()),
		kgo.BatchCompression(kgo.SnappyCompression()),
		kgo.ClientID("my-client-id"),
		kgo.MaxVersions(kversion.V2_6_0()),
		kgo.WithLogger(&loggerForFranz{log, kgo.LogLevelInfo}),
	)

and use it to issue MetadataRequest and  ListOffsetsRequest, then set client.AssignPartitions() based on response and consume records from topic. This works ok. I leave an background goroutine PollFetches-ing.
Using the same client I also try to produce record to topic:
	err := client.Produce(ctx,
		&kgo.Record{
			Topic: "my.topic.name",
			Key:   key,
			Value: value,
		},
		func(r *kgo.Record, e error) {
			if e != nil {
				// log error
			}
		})

this logs error from promise: "CLUSTER_AUTHORIZATION_FAILED: Cluster authorization failed.".
stack trace
<EMAIL_ADDRESS>: 147 : github.com/twmb/franz-go/pkg/kgo.(*Client).finishRecordPromise
<EMAIL_ADDRESS>: 1184 : github.com/twmb/franz-go/pkg/kgo.(*recBuf).lockedFailAllRecords
<EMAIL_ADDRESS>: 1166 : github.com/twmb/franz-go/pkg/kgo.(*recBuf).failAllRecords
<EMAIL_ADDRESS>: 305 : github.com/twmb/franz-go/pkg/kgo.(*sink).drain

and logs from the lib:
initializing producer id
producer id initialization errored
map[err:CLUSTER_AUTHORIZATION_FAILED: Cluster authorization failed.]
InitProducerID or AddPartitionsToTxn, failing producer id
map[err:CLUSTER_AUTHORIZATION_FAILED: Cluster authorization failed.]
ignoring a fail producer id request due to current id being different
map[current_epoch:-1 current_err:CLUSTER_AUTHORIZATION_FAILED: Cluster authorization failed. current_id:-1 fail_epoch:0 fail_err:CLUSTER_AUTHORIZATION_FAILED: Cluster authorization failed. fail_id:0]

What I'm doing wrong?

Hi!
This client by default opts in to idempotent writes and currently has no way of disabling this. I've been intending to add an option to disable this because not everybody wants this, and I'll do that shortly.
Until that is done, idempotent writes require
operation=IDEMPOTENT_WRITE, permissionType=ALLOW, resource: cluster
Is this permission missing, and if so, is it possible to add it?

I think the permission is missing, what I see in kafka-ui under ACLS for the topic is just "ALLOW DESCRIBE READ WRITE"... have to talk to the admin to see can it be added.

Hi there! I introduced an option to disable idempotent writes with https://github.com/twmb/franz-go/commit/3e6bcc3. I'm going to close this issue, please reopen if this does not fix things.

In the meantime my admin tweaked something in the user rights and producing into the topic started to work.
However the code which worked before now fails to read topic offsets - it (kmsg.NewListOffsetsRequest()) gets -1 as both oldest and newest offset for the topic's single partition...?
Still the same with latest version 0.6.10

Interesting. Are you referring to the request in the consumer.go code, or are you issuing that request manually?
ListOffsets uses -1 for offsets if there is an error, and the ErrorCode field will be nonzero if so. Otherwise, -1 is also used if the requested partition does not exist.
If you're issuing the request manually, what does the ErrorCode field say?
If this is from the consumer.go code, if the error is not retriable, the error is returned in a TopicPartition in PollFetches. If the error is retriable, the load continues to retry and you wouldn't see an error. If this was from no error but ListOffsets returned -1, I think the partition is avoided being fetched.

Ahh whoops I meant -2 (earliest offsets). Cool, glad this is solved for you!


[UWP][Image Rendering] Revisit image async loading

Platform

UWP

Version of SDK
main
Details
Async tasks used for Image loading in UWP should be refactored. As XamlBuilder owns the control over all async calls, small changes can break the detection of the moment when images finish loading, for example, this issue blocks PRs as #3113 . By going back to refactor this issue we can tackle other topics such as opening the functionality to wait for image loads in the nuget package version and the best way to expose it (adding an images loaded event vs having this functionality as part of an async call).

guessing that this will be addressed by #6323


OpenSCAP json tailoring (HMS-4462)

This depends on https://github.com/osbuild/osbuild-composer/pull/4272

sorry - did a rebase - I hope that was ok :-|

sorry - did a rebase - I hope that was ok :-|

No worries, I've just fixed and rebased again


Build fails when a controller or template sends a HTTP header?

In a feed.php controller:
header::type('application/atom+xml');

Response when building the full list of pages which does contain several feed.txt pages is HTTP 404 with Content-Type: application/atom+xml; charset=UTF-8 and no body.

Looks like redirection headers are being sent too.

It seems that:

Kirby itself respects the $headers=false param for Kirby::render
User code (in templates, controllers, page methods, plugins etc.) using the header class, PHP's header(), the go() helper, etc. will not follow this param (quite logically).

This can’t be changed without changes to user code (header_remove() in the plugin code has no effect whatsoever…), so the question seems to be: what’s a good way (current or future) with Kirby to know if a page is being rendered with $headers=true or not, or maybe what’s a good way to defer sending headers to Kirby.


Remove dependence of HarbourBridge UI on static assets

Expected Behavior
HarbourBridge should be deployable to customers as a binary without needing additional configuration files or static assets which need to be distributed outside of the binary.
Actual Behavior
The web UI currently depends on static assets whose relative paths are specified in the code. Due to this it can only be run when the binary is placed alongside (in the same directory) as the source code. The static assets that are used for rendering the web UI are not packaged with the binary
Steps to Reproduce the Problem

Build HB binary go build
Run harbourbridge from any path other than the directory in which the binary is present OR copy the binary to any other directory.
Run the UI using the --web flag.
localhost:8080 displays a 404 page not found error.

Sample ref on how to embed static assets -
https://harsimranmaan.medium.com/embedding-static-files-in-a-go-binary-using-go-embed-bac505f3cb9a

Fixed in #320


l7 policies do not work in gke

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Environment:

gke (with dataplane v2 & auto-repair disabled)
cilium 1.13.0

Steps to reproduce:

make sure no CNP (cilium network policies) are added.

$ kubectl get cnp -A
No resources found

deploy deathstar, spaceship and xwing from https://github.com/cilium/star-wars-demo/blob/master/01-deathstar.yaml and https://github.com/cilium/star-wars-demo/blob/master/02-xwing.yaml. In this case I changed replicas=1 but it does not matter.

kubectl get pods -o wide
NAME                         READY   STATUS    RESTARTS   AGE   IP             NODE                              NOMINATED NODE   READINESS GATES
deathstar-7bd4f99db8-q46nx   1/1     Running   0          5s    <IP_ADDRESS>    gke-infra-gke-gp1-a4e93c2c-00d5   <none>           <none>
spaceship-69789b6f5-g4tjg    1/1     Running   0          8d    <IP_ADDRESS>   gke-infra-gke-gp1-a4e93c2c-scr5   <none>           <none>
xwing-5f4b9fdc6-nr4lj        1/1     Running   0          8d    <IP_ADDRESS>    gke-infra-gke-gp1-a4e93c2c-00d5   <none>           <none>

Apply l7_policy https://github.com/cilium/star-wars-demo/blob/master/policy/l7_policy.yaml

$ kubectl get cnp
NAME                       AGE
deathstar-api-protection   4s

$kubectl describe cnp
Name:         deathstar-api-protection
Namespace:    default
Labels:       <none>
Annotations:  <none>
API Version:  cilium.io/v2
Kind:         CiliumNetworkPolicy
Metadata:
  Creation Timestamp:  2023-03-31T17:14:54Z
  Generation:          1
  Managed Fields:
    API Version:  cilium.io/v2
    Fields Type:  FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          .:
          f:kubectl.kubernetes.io/last-applied-configuration:
      f:spec:
        .:
        f:endpointSelector:
          .:
          f:matchLabels:
            .:
            f:class:
            f:org:
        f:ingress:
    Manager:         kubectl-client-side-apply
    Operation:       Update
    Time:            2023-03-31T17:14:54Z
  Resource Version:  5510654
  UID:               5af586dd-98ae-4d4a-a5d0-f8fae144919f
Spec:
  Endpoint Selector:
    Match Labels:
      Class:  deathstar
      Org:    empire
  Ingress:
    From Endpoints:
      Match Labels:
        Class:  spaceship
    To Ports:
      Ports:
        Port:      80
        Protocol:  TCP
      Rules:
        Http:
          Method:  GET
          Path:    /v1/
          Method:  POST
          Path:    /v1/request-landing$
          Headers:
            X-Has-Force: True
          Method:  PUT
          Path:    /v1/exhaust-port$
Events:            <none>

attack from xwing to deathstar

kubectl exec -ti xwing-5f4b9fdc6-nr4lj -- curl -XPUT -H 'X-Has-Force: True' deathstar.default.svc.cluster.local/v1/exhaust-port

Actual:  curl timed out
curl: (56) Recv failure: Connection reset by peer
command terminated with exit code 56

Expected:
Panic: deathstar exploded

goroutine 1 [running]:
main.HandleGarbage(0x2080c3f50, 0x2, 0x4, 0x425c0, 0x5, 0xa)
        /code/src/github.com/empire/deathstar/
        temp/main.go:9 +0x64
main.main()
        /code/src/github.com/empire/deathstar/
        temp/main.go:5 +0x85

Cilium Version
cilium-cli: v0.13.1 compiled with go1.20 on linux/amd64
cilium image (default): v1.13.0
cilium image (stable): v1.13.1
cilium image (running): v1.13.0
Kernel Version
kubectl exec -n kube-system cilium-rfl92 -- uname -a
Defaulted container "cilium-agent" out of: cilium-agent, config (init), mount-cgroup (init), apply-sysctl-overwrites (init), mount-bpf-fs (init), wait-for-node-init (init), clean-cilium-state (init)
Linux gke-infra-gke-gp1-a4e93c2c-scr5 5.4.0-1086-gke #93-Ubuntu SMP Mon Oct 17 13:32:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Kubernetes Version
Client Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.0", GitCommit:"4ce5a8954017644c5420bae81d72b09b735c21f0", GitTreeState:"clean", BuildDate:"2022-05-03T13:46:05Z", GoVersion:"go1.18.1", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.4
Server Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.14-gke.1800", GitCommit:"1eab5b8da4acab130c72aea21eb7ed3e96523ca2", GitTreeState:"clean", BuildDate:"2022-12-07T09:32:46Z", GoVersion:"go1.17.13b7", Compiler:"gc", Platform:"linux/amd64"}
Sysdump
cilium-sysdump-20230331-101840.zip
Relevant log output
gke cluster

kubectl get pods -n kube-system
NAME                                         READY   STATUS    RESTARTS     AGE
cilium-mt88m                                 1/1     Running   0            2d19h
cilium-node-init-ppjp5                       1/1     Running   0            2d19h
cilium-node-init-q9clr                       1/1     Running   0            2d19h
cilium-node-init-w2xfc                       1/1     Running   0            2d19h
cilium-operator-7cd45877b-st5m5              1/1     Running   0            2d19h
cilium-rfl92                                 1/1     Running   0            2d19h
cilium-tlqvf                                 1/1     Running   0            2d19h
event-exporter-gke-5dc976447f-dp5q6          2/2     Running   0            8d
fluentbit-gke-4c9rj                          2/2     Running   0            8d
fluentbit-gke-56bq6                          2/2     Running   2 (8d ago)   8d
fluentbit-gke-jz2kz                          2/2     Running   0            8d
gke-metrics-agent-4xfcz                      1/1     Running   0            7d17h
gke-metrics-agent-9xh9k                      1/1     Running   0            7d17h
gke-metrics-agent-gsdld                      1/1     Running   0            7d17h
hubble-relay-6dc9cb87d5-6tdtl                1/1     Running   0            2d19h
kube-dns-7d44cdb5d5-4s9xt                    4/4     Running   0            8d
kube-dns-7d44cdb5d5-5xmcm                    4/4     Running   0            8d
kube-dns-autoscaler-fbc66b884-8gl76          1/1     Running   0            8d
kube-proxy-gke-infra-gke-gp1-a4e93c2c-00d5   1/1     Running   1 (8d ago)   8d
kube-proxy-gke-infra-gke-gp1-a4e93c2c-djjh   1/1     Running   0            8d
kube-proxy-gke-infra-gke-gp1-a4e93c2c-scr5   1/1     Running   0            8d
metrics-server-v0.5.2-866bc7fbf8-nnt5k       2/2     Running   0            8d
netd-bwvgz                                   1/1     Running   0            8d
netd-hbx8p                                   1/1     Running   0            8d
netd-kj2v5                                   1/1     Running   1 (8d ago)   8d
pdcsi-node-h84km                             2/2     Running   0            8d
pdcsi-node-jz7ls                             2/2     Running   0            8d
pdcsi-node-t48hq                             2/2     Running   2 (8d ago)   8d

hubble observe on default/xwing-5f4b9fdc6-nr4lj
$ kubectl exec -it cilium-mt88m -n kube-system  -- hubble observe --pod default/xwing-5f4b9fdc6-nr4lj -f

Mar 31 17:40:54.248: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) -> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-stack FORWARDED (TCP Flags: SYN)
Mar 31 17:40:54.248: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: SYN)
Mar 31 17:40:54.249: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) -> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) policy-verdict:L3-L4 INGRESS ALLOWED (TCP Flags: SYN)
Mar 31 17:40:54.249: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) -> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-proxy FORWARDED (TCP Flags: SYN)
Mar 31 17:40:54.249: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <- default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-endpoint FORWARDED (TCP Flags: SYN, ACK)
Mar 31 17:40:54.249: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) -> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-stack FORWARDED (TCP Flags: ACK, PSH)
Mar 31 17:40:54.249: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK, PSH)
Mar 31 17:40:54.249: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) -> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-stack FORWARDED (TCP Flags: ACK)
Mar 31 17:40:54.249: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK)
Mar 31 17:40:54.453: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK, PSH)
Mar 31 17:40:54.661: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK, PSH)
Mar 31 17:40:55.077: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK, PSH)
Mar 31 17:40:55.269: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK)
Mar 31 17:40:55.909: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK, PSH)
Mar 31 17:40:57.285: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK)
Mar 31 17:40:57.577: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK, PSH)
Mar 31 17:41:00.869: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) -> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-stack FORWARDED (TCP Flags: ACK, PSH)
Mar 31 17:41:00.869: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK, PSH)
Mar 31 17:41:01.381: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <- default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-endpoint FORWARDED (TCP Flags: SYN, ACK)
Mar 31 17:41:01.381: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK)
Mar 31 17:41:07.525: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) -> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-stack FORWARDED (TCP Flags: ACK, PSH)
Mar 31 17:41:07.525: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK, PSH)
Mar 31 17:41:09.573: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <- default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-endpoint FORWARDED (TCP Flags: SYN, ACK)
Mar 31 17:41:09.573: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK)
Mar 31 17:41:20.837: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) -> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-stack FORWARDED (TCP Flags: ACK, PSH)
Mar 31 17:41:20.837: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK, PSH)
Mar 31 17:41:25.701: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <- default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-endpoint FORWARDED (TCP Flags: SYN, ACK)
Mar 31 17:41:25.701: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK)
Mar 31 17:41:47.973: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) -> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-stack FORWARDED (TCP Flags: ACK, PSH)
Mar 31 17:41:47.973: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK, PSH)
Mar 31 17:42:41.221: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) -> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-stack FORWARDED (TCP Flags: ACK, PSH)
Mar 31 17:42:41.221: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) DBG_CAPTURE_DELIVERY VERDICT_UNKNOWN (TCP Flags: ACK, PSH)
Mar 31 17:42:41.222: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) -> default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-proxy FORWARDED (TCP Flags: ACK, PSH)
Mar 31 17:42:41.222: default/xwing-5f4b9fdc6-nr4lj:56450 (ID:54246) <- default/deathstar-7bd4f99db8-q46nx:80 (ID:42441) to-endpoint FORWARDED (TCP Flags: RST)

### Anything else?

_No response_

### Code of Conduct

- [X] I agree to follow this project's Code of Conduct

My mistake, I mis-read the issue.

@squeed  do we have anyone to take a look at this issue?

Did you try with the latest v1.13.x?

Not yet, will try in a couple of days.
Any reason why later version could work?

I can atest that L7 tests fail using dataplane V1 + GKE.  Research to understand whether it can work has been difficult. I have found two documents wrt to Calico (what is typically preinstalled as the CNI for GKE for V1 dataplane).  both are talk about CNI chaining (i.e. that both calico and cilium will exist I assume). The first link seems only "valid" for < 1.13 releases https://docs.cilium.io/en/v1.12/gettingstarted/cni-chaining-calico/ and clearly calls out that CNI chaining causes issues for Layer 7 Policy.  and refers to https://github.com/cilium/cilium/issues/12454.  I had hoped that maybe in 1.13 they didn't need the CNI chaining documenation as they fixed this somehow, but that isn't my experience at the moment.

also, let me know I can post more information wrt to my configuration and the error message if that is helpful.

The issue still there in gke dpv1 & cilium 1.15.1


Shorten max-services to max-serv to avoid line length issue

Description
Shorten max-services to max-serv to avoid line length issue. Can now successfully create 5000 services in a single namespace.
Fixes

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_kubeburner
Pass
0
00:22:56


Migrate to siwe-viem (instead of vanilla SIWE)

Requires #24

Related to https://github.com/kleros/atlas/issues/13?


Mathjax cdn update

MathJax's CDN is shutting down I've changed to cdnjs as suggested in the article. Note that:

Version number must be changed manually (there's no latest anymore.)
I've changed from HTTP to HTTPS which is I think a safer default.

@abeaumont thanks!


Cyber_Security: 1.9.7_Types_of_Hackers

Description 📜
It describes who is a hacker and what are the different types of hackers
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #148

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Cyber Security

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@0shivamh You did great work!
But can you add some more content, you wrote very less just1-2 lines, like Blue-hat, Green-hat, Red-hat.

Okay, @Anant1711 , I'll do it

@0shivamh You did great work!
But can you add some more content, you wrote very less just1-2 lines, like Blue-hat, Green-hat, Red-hat.

Okay, @Anant1711 , I'll do it

Hey @Anant1711, it is fine now, or have to add more content.


Rectangular matrices

What should I do if I have rectangular matrices? Would I have to implement it myself?

Unfortunately rectangular matrices are not implemented yet in nalgebra. I tried a few months ago but the macro system was not powerful enough do to this in a clean way. I believe this should be possible now.

Though you can use a DMat for rectangular matrices, but this will allocate (its size is not fixed at compile time).

Fixed by #218. Rectangular matrices of any dimension are now supported.


Fix SendTritium macro SENDTRITIUM_EXPOSE_VARS so that it can compile

Describe the bug
SendTritium currently has a macro SENDTRITIUM_EXPOSE_VARS that assists during debugging by exposing variables related to hardware input and hiding the code for reading them on hardware. However, SENDTRITIUM_EXPOSE_VARS currently doesn't work since #define-ing it creates compile errors from externing static variables and defining unused variables.
To Reproduce
Steps to reproduce the behavior:

Define SENDTRITIUM_EXPOSE_VARS in SendTritium on the master branch
Compile leader (main or a test)

Expected behavior
Defining SENDTRITIUM_EXPOSE_VARS should disable CAN messages and reading inputs and instead allow you to set/inspect variables in a test file by externing them in SendTritium.h.

Seems to be fine in dev/code_cleanup

Cool, I guess the changes there fix it! I'll link this to the code cleanup PR.


received invalid data

Whenever go-whatsapp wants to read the server's answer after sending a message (or a read acknowledgement), the connection fails with
error processing data: received invalid data

and a reconnect is needed. This ErrInvalidWsData originates from https://github.com/Rhymen/go-whatsapp/blob/3f56542bf70be8ff87e23880ef08ec4b88dd1aa2/read.go#L59. Messages are sent to the contact, though.
This affects at least my own application and the echo example.

Can you please check if the same error occoured before I merged the latest PR? I tagged the version as v0.0.1 before I merged

Yes, I just tried with the current master. Strangely enough, the sendTextMessages example is not affected.

The go.mod of the examples referenced an old version of go-whatsapp. I changed them now to use the latest version.

Thanks. Your changes at 3fb29d5 seem to have been reverted at 7f76648a.
Also the examples won't work out of the box as they still use RestoreSession rather than RestoreWithSession.
Related to the original question, I managed to pull some debug output from the communication. This is what is encountered up in processReadData:
Valid data: []string{"!", "1552512078460"}
Valid data: []string{"1552512078.--0", "REDACTED_JSON"}
Valid data: []string{"s1", "[\"Conn\",REDACTED_JSON]"}
Valid data: []string{"d1fc256c5766a1c6.--1f", REDACTED_BINARY}
Valid data: []string{"s2", "[\"Blocklist\",{\"id\":1,\"blocklist\":[]}]"}
Valid data: []string{"1552512078.--1", "{\"status\":200}"}
Valid data: []string{"s3", "[\"Stream\",\"update\",true,\"0.3.2386\"]"}
Valid data: []string{"s4", "[\"Props\","REDACTED_JSON"]"}
Valid data: []string{"preempt-d1fc256c5766a1c6.--20", "REDACTED_BINARY"}
Valid data: []string{"d1fc256c5766a1c6.--21", "REDACTED_BINARY"}
Valid data: []string{"!", "1552512098856"}
Valid data: []string{"d1fc256c5766a1c6.--24", "REDACTED_BINARY"}
Invalid data: []string{"2BA5323571AD0567DD4D", ""}
error caught in handler: error processing data: received invalid data
Valid data: []string{"2BA5323571AD0567DD4D", "REDACTED_JSON"}
echoed message '@echo test6' to user<EMAIL_ADDRESS>Valid data: []string{"s7", "REDACTED_JSON"}
Valid data: []string{"!", "1552512128854"}

It seems like the function is fed an empty string. The function is then re-invoked with valid data.
I was under the impression errors fed to the handler are critical and need to be coped with. Seems I was mistaken. In my application, I now ignore this error silently.


Why it does that
Briefly state why the change was necessary.
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes # <-- insert Issue number if one exists

Codecov Report
Patch and project coverage have no change.

Comparison is base (9ac4976) 72.86% compared to head (748b496) 72.86%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #67   +/-   ##
=======================================
  Coverage   72.86%   72.86%
=======================================
  Files          20       20
  Lines         479      479
  Branches       17       17
=======================================
  Hits          349      349
  Misses        129      129
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Connecting without sql authentication?

At the company I work at we use active directory to authorize servers and machines that talk to SQL server. This is for integration services and security. So I won't be able to set up a username and password within SQL security profiles. Does the API support connection strings that work with active directory or native client? If so I couldn't find out how in the docs. I am currently using your code with msnodesql. Thanks.

Please see this section of the docs. There is an option for customizing connection string used by msnodesql.

I took a glimpse over that earlier. I will check it out again. Just didn't know if when specifying the configuration I could have an option to set integrated security true or SSPI.

@baconface It's worth noting, that the MS node adapter isn't well supported, will only work in windows, and has a build dependency against your node version and platform (32/64-bit), and windows platform (32/64-bit).

Thanks @tracker1 . I was aware of the native bindings but in our case we don't suffer from those limitations. As it is an internal service being ran on a Windows 2012 Server and is not meant for consumer use or non-Win machines. Right now I am using Edge.js and the Edge.js SQL packages. It works for this project. Especially since we are a JavaScript/C# house. But someday I dream of dropping Edge.js since I feel like it is a round-about/dirty way of talking to the database. Our end user apps talk to the database using a Web API C# project and I also considered making our internal node.js services do the same. But this project still looks real nice and I am going to test it out with the other two TDS projects and see if I can get it going as it would allow me to write our internal service apps a lot quicker.


Allow filtering command in ebpf code

What type of PR is this?
/kind feature
What this PR does / why we need it:
As discussed in https://github.com/kubernetes-sigs/security-profiles-operator/pull/1497#discussion_r1116870791, we now filter the command out in the CLI to avoid recording the whole system.
Which issue(s) this PR fixes:

Part of https://github.com/kubernetes-sigs/security-profiles-operator/issues/1482
Requires https://github.com/kubernetes-sigs/security-profiles-operator/pull/1502
Does this PR have test?
None

Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

Codecov Report

Merging #1503 (007300a) into main (0d896dd) will increase coverage by 0.25%.
The diff coverage is 84.84%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1503      +/-   ##
==========================================
+ Coverage   43.85%   44.10%   +0.25%
==========================================
  Files          51       50       -1
  Lines        5724     5736      +12
==========================================
+ Hits         2510     2530      +20
+ Misses       3093     3085       -8
  Partials      121      121

I'm going to rebase this once https://github.com/kubernetes-sigs/security-profiles-operator/pull/1502 got merged

Had to rebase once again. PTAL @kubernetes-sigs/security-profiles-operator-maintainers


chore: move to cabinjs

Pull Request Test Coverage Report for Build 143776018

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.136%

Totals

Change from base Build 143767600:
0.0%

Covered Lines:
45

Relevant Lines:
52

💛  - Coveralls


What happens to the test Bitcoin if stuck on Step 4?

I got an error on Step 4 - Error submitting proof.

I have also read the previous threads about how this is an issue on testnet Bitcoin side.
However...

what is the solution?
what happens to my testnet Bitcoin?
where is it?
if I cannot get pass Step 4, what should I do?
can I get my testnet Bitcoin back? It took a lot of time and effort to accumulate 1 testnet Bitcoin.

I have the same problem, lost 2 BTC so far due to the same problem:
https://dapp.test.tbtc.network/deposit/0x55011DA991a3BfBD8a24CfaDdA112755057B7831/prove
https://dapp.test.tbtc.network/deposit/0x67dB4AcE460d323d1ab4c4eF97236D267879DE1D/prove

can I get my testnet Bitcoin back? It took a lot of time and effort to accumulate 1 testnet Bitcoin.

Both of you, save the links and proof send them in in the redemption form - we can make sure they're handled on the other side.

Thanks Matt. Will do.


Does not compile feature in pt-BR when the word contains accents

Reqnroll Version
1.0.0
Which test runner are you using?
xUnit
Test Runner Version Number
2.5.6
.NET Implementation
.NET 8.0
Test Execution Method
Visual Studio Test Explorer
Content of reqnroll.json configuration file
{
"$schema": "https://schemas.reqnroll.net/reqnroll-config-latest.json",
"language": {
"feature": "pt-BR"
}
}
Issue Description
Does not compile feature in pt-BR when the word contains accents:
Funcionalidade: Calculator
Simple calculator for adding two numbers
@mytag
Cenário: Add two numbers
Dado the first number is 50
E the second number is 70
Quando the two numbers are added
Então the result should be 120
Steps to Reproduce
Write a feature using gherkin reserved words in the pt-BR language:
compile the project
and check if an error occurs:

However, there is no error if I remove accents, but it is a spelling error in this language:

Link to Repro Project
https://github.com/tulavalle/ReqnrollSelenium.git

@tulavalle This is strange, because the parsing of the files is not done by Reqnroll itself, but by the Cucumber Gherkin parser. Was this working with SpecFlow before or is this a new project?

Hi @gasparnagy ! It was a new project, I ran tests here and restarted the machine, it seems to have returned to normal:

It even normalized the developer console, after installing the Reqnroll extension it was no longer opening. I will create a new project in a few days and see if the same thing happens. Any eventuality I will share here!
Note: The same did not happen when migrating projects, I migrated large projects and it was very easy and uneventful. I needed to revert to Specflow because I depend on LivingDoc to publish the documentation for the LivingDoc extension on Azure. As soon as this functionality is implemented I will definitely migrate, congratulations on the work with Reqnroll, Specflow is very good and ReqnRoll is even better!


Update dependencies to use graphql, graphene > v3

Oh, so this library is not maintained anymore? 😲

Let's provide these details in the description


KOGITO-8257: Create an integration test for Dashbuilder VSCode Extension

@barboras7 can you please try again?


Add py.typed file to package for PEP-561 compliance

Per PEP-561, packages that include type information that can be consumed
by other libraries should distribute a py.typed file. This tells mypy
and other tools to use type information shipped with the library.
This requires moving distro from a single module file to a package so
that it can ship data files. The original distro.py was not altered.
Only the __init__.py and __main__.py were added to facilitate being used
as a package. By keeping distro.py unaltered, this allows projects to
continue to vendor the file into their project without as before without
any modifications.
For details on PEP-561, see:
https://www.python.org/dev/peps/pep-0561/
For details on mypy using py.typed, see:
https://mypy.readthedocs.io/en/stable/installed_packages.html#creating-pep-561-compatible-packages

Please resolve conflicts.
I'm also wondering if there are cases in which people already download directly, which will now break.
Maybe there's a way to redirect?

@jdufresne This PR appears nearly complete, just pinging you in case you still intend to land it.

@jdufresne Looks like a recent merge introduced a rebase issue, could you handle that @jdufresne?

@sethmlarson No problem. Rebased.


[DO NOT MERGE][stdlib] Keep Dictionary storage at least 25% full when possible

This is an experimental implementation of shrinking Dictionary storage after removals.
It isn't clear this is worth doing, and the bulk of the actual work towards landing it would be in benchmarking and adding test coverage, which is not at all done. However, the implementation is an interesting exercise in library evolution.
It's almost possible to make this feature deploy back to earlier OSes -- except Set/Dictionary doesn't record the reserved capacity in 5.0 & 5.1, and that arguably rules out shrinking. (We'd also have to make the minimum load factor part of the ABI, but that's less of an issue.)
rdar://problem/18114559

Just to see if we have adequate benchmarks for Dictionary removals:
@swift-ci benchmark

@swift-ci test

Performance: -O

Regression
OLD
NEW
DELTA
RATIO

DictionaryRemove
3150
7460
+136.8%
0.42x

DictionaryRemoveOfObjects
21700
27900
+28.6%
0.78x

ObjectiveCBridgeStubFromNSDateRef
3760
4190
+11.4%
0.90x (?)

Improvement
OLD
NEW
DELTA
RATIO

String.data.Medium
98
86
-12.2%
1.14x (?)

CharacterLiteralsLarge
108
97
-10.2%
1.11x (?)

String.data.LargeUnicode
106
96
-9.4%
1.10x (?)

CharacterLiteralsSmall
340
317
-6.8%
1.07x (?)

Code size: -O

Regression
OLD
NEW
DELTA
RATIO

DictionaryRemove.o
15985
16337
+2.2%
0.98x

Performance: -Osize

Regression
OLD
NEW
DELTA
RATIO

DictionaryRemove
5640
15100
+167.7%
0.37x

DictionaryRemoveOfObjects
29900
48800
+63.2%
0.61x

Improvement
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeStubFromNSStringRef
207
175
-15.5%
1.18x (?)

ObjectiveCBridgeFromNSSetAnyObjectToString
78000
71500
-8.3%
1.09x (?)

NSStringConversion.UTF8
831
776
-6.6%
1.07x (?)

Code size: -Osize

Improvement
OLD
NEW
DELTA
RATIO

DictionaryRemove.o
10833
10657
-1.6%
1.02x

Performance: -Onone

Regression
OLD
NEW
DELTA
RATIO

DictionaryRemove
13230
24070
+81.9%
0.55x

DictionaryRemoveOfObjects
44400
69300
+56.1%
0.64x

Improvement
OLD
NEW
DELTA
RATIO

ArrayOfPOD
1146
1064
-7.2%
1.08x (?)

Code size: -swiftlibs

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

A 2x slowdown matches how insertion behaves when capacity was reserved vs when it wasn't.
Some of it is probably spent on runtime availability checks that we could perhaps eliminate.

Build failed
Swift Test OS X Platform
Git Sha - a2e2f9bb287d9c6421840b1cb3ac0344e916c83f

Can one of the admins verify this patch?

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

How to change the base branch: (Link)
More detail about the branch update: (Link)


v2.0.0 RC

@kayladnls @localheinz @hannesvdvreken

:+1: glad it's ready for tagging a release candidate! Good work.

@codeguy be interested to get your opinions on this from the perspective of interoperability if you have any time?

@hannesvdvreken https://packagist.org/packages/league/route#2.0.0-RC1

@philipobenito c/p the example docs and /prefix/example isn't routing correctly. Getting League\Route\Http\Exception\NotFoundException

Could you check out your REQUEST_URI for me please?
On Mon, Oct 5, 2015 at 4:38 PM, Kayla Daniels<EMAIL_ADDRESS>wrote:

@philipobenito c/p the example docs and /prefix/example isn't routing correctly. Getting League\Route\Http\Exception\NotFoundException
Reply to this email directly or view it on GitHub:
https://github.com/thephpleague/route/pull/73#issuecomment-145574147

Zend\Diactoros\Uri Object
(
    [allowedSchemes:protected] => Array
        (
            [http] => 80
            [https] => 443
        )

    [scheme:Zend\Diactoros\Uri:private] =>
    [userInfo:Zend\Diactoros\Uri:private] =>
    [host:Zend\Diactoros\Uri:private] =>
    [port:Zend\Diactoros\Uri:private] =>
    [path:Zend\Diactoros\Uri:private] => /prefix/example
    [query:Zend\Diactoros\Uri:private] =>
    [fragment:Zend\Diactoros\Uri:private] =>
    [uriString:Zend\Diactoros\Uri:private] =>
)

nvm @philipobenito I think I got it working.

Ah cool, what was the issue?
On Mon, Oct 5, 2015 at 6:23 PM, Kayla Daniels<EMAIL_ADDRESS>wrote:

nvm @philipobenito I think I got it working.
Reply to this email directly or view it on GitHub:
https://github.com/thephpleague/route/pull/73#issuecomment-145604363

I was testing in on cli by setting the uri on the request object. Which fails because there's no schema or host.

I would strongly suggest rebasing this into a smaller number of commits and using The Rules for your commit messages.

I'm also strongly against the idea of ContainerAwareTrait and any code that calls new outside of factories. This library would be much better if it did not depend on league/container.

@shadowhand

ContainerAware* are for use within factories.
The whole package is essentially a factory where in a lot of cases objects have to be created.
How is the quality of the library diminished by the inclusion of something that the default strategies depend on? The point is that it is already more difficult than it was to use out of the box, I'm not gonna make it even more so.

If you start with the assumption that the strategies need the container, then you will not be able to see how to make it work without them. This is the same general problem that all containers have... "when your own tool is a hammer..."

@shadowhand those points are pretty moot, the router does not force you to do anything, however, it provides an interface to get people started. Remember, you cannot assume that everybody knows what you know. With this in mind, route provides a little convenience out of the box. Otherwise, it would be unusable without a much higher barrier to entry.
a) you do not have to use league/container, b) you do not have to use any container. I am not forcing you to use a container.

@philipobenito of course you are... composer.json says that league/container is required to use this package.

@shadowhand out of the box it is required for the intended use, however, it is extensible enough for you to not have to actually use it. I'll reiterate again, this is for convenience to users who may not be comfortable enough to build their own strategies etc.
As for users such as yourself who don't want the library to be opinionated in any way of how it builds class based controllers, my question would be, why are you using league/route instead of just using FastRoute?

Primary because league/route makes it easier to work with FastRoute. I wanted to use this package for spark but we ended up rolling a custom router because of the container dependency (and because it wasn't PSR-7 ready).

@shadowhand well that is the point, it makes it a little easier, if it didn't there would be no reason for this package to exist.

In which case, why not choose to minimize the dependencies and avoid the container dependency entirely?

@shadowhand because it also aims to encourage people to use dependency injection throughout their projects. That is the point of the strategies, and each strategy works out of the box. However, very very easy to build your own strategy that does not use a container. This is something I've said to you before, your use cases are not necessarily the same as other peoples, the intended use of the default set up of this package is with the container. However, I understand some people do not want to use one, therefore, it is very easy not to.

If the container is not required than it should be a suggests dependency instead of requires.

It is required... out of the box... but... you don't have to use it if you don't want to, it really is that simple! It provides a tool that is required for the package to work as is. However, and I don't know how many times I can reiterate this, if you are of the opinion you don't want to use it, then don't. Would you really say that removing the container because you don't want to use it would be a better option for the package than leaving it in for the much larger user base who do want to use it?

Libraries that do not require a container are better for everyone. I guess you don't see it that way.

@shadowhand I both agree and disagree. This library, by default, is opinionated on how objects are built (again, it allows you not to adhere to this opinion), therefore, a container is a good idea, however, that is not the issue here, the issue is barrier to entry. By excluding it, I am alienating anybody who is not comfortable setting that stuff up for themselves.

So just to end the above conversation as it is in no way productive.
Including the container by default is a conscious decision to allow the built in strategies to be used out of the box. This is a trade-off between doing some of the work for the user and doing none of the work for the user. It is an option for people who do not want to use a full stack framework but do not want to do all the heavy lifting themselves. It sits somewhere in the middle of Laravel or similar and FastRoute or similar without being considered a micro framework such as Slim or Silex.
By removing the container, along with that I have to remove the current strategies that are included or change them to just invoke a callable. At that point, there is no need for this package to exist as that is what FastRoute does. It is a trivial extension of a non-trivial package to lower the barrier of entry and promote package driven development and a couple of design patterns/methods that will make your life a little easier while allowing you to write good code.
All this being said, as opinionated as the package is out of the box, it is extensible enough for you to ignore the defaults completely and do as little or as much as you like, if you want it to have no opinions at all, my advice would be to build something similar that works for you on top of FastRoute or similar.
A specific example of this would be @codeguy considering league/route for the new version of Slim. It has functionality that is not needed by Slim, therefore, a decision was made to replace it with FastRoute directly. This would be my advice to anybody who is so strongly opinionated about how the dispatch of a controller should be handled. Otherwise, build a strategy, ignore the container completely, and it will not be used.

To be honest I would rather see container-interop as a dependency rather
than league/container, because container-interop provides interfaces in a
PSR-like style, and thus leaving implementation to be chosen. While
league/container isn't a very heavy DI container library, it does
"contaminate" the dependency graph with an implementation that the
developer might not want to use.
I believe that a library, especially one like this, should only depend on
bare necessities.
On Oct 12, 2015 5:16 PM, "Phil Bennett"<EMAIL_ADDRESS>wrote:

So just to end the above conversation as it is in no way productive.
Including the container by default is a conscious decision to allow the
built in strategies to be used out of the box. This is a trade-off between
doing some of the work for the user and doing none of the work for the
user. It is an option for people who do not want to use a full stack
framework but do not want to do all the heavy lifting themselves. It sits
somewhere in the middle of Laravel or similar and FastRoute or similar
without being considered a micro framework such as Slim or Silex.
By removing the container, along with that I have to remove the current
strategies that are included or change them to just invoke a callable. At
that point, there is no need for this package to exist as that is what
FastRoute does. It is a trivial extension of a non-trivial package to lower
the barrier of entry and promote package driven development and a couple of
design patterns/methods that will make your life a little easier while
allowing you to write good code.
All this being said, as opinionated as the package is out of the box, it
is extensible enough for you to ignore the defaults completely and do as
little or as much as you like, if you want it to have no opinions at all,
my advice would be to build something similar that works for you on top of
FastRoute or similar.
A specific example of this would be @codeguy https://github.com/codeguy
considering league/route for the new version of Slim. It has functionality
that is not needed by Slim, therefore, a decision was made to replace it
with FastRoute directly. This would be my advice to anybody who is so
strongly opinionated about how the dispatch of a controller should be
handled. Otherwise, build a strategy, ignore the container completely, and
it will not be used.
—
Reply to this email directly or view it on GitHub
https://github.com/thephpleague/route/pull/73#issuecomment-147430836.

It's probably not very useful to add yet another voice to this discussion :stuck_out_tongue:

To be honest I would rather see container-interop as a dependency rather than league/container

Yes and no. I agree, except for one thing: There is a call method on league/container's implementation that is used to auto inject stuff. By using that, it needs league/container as a hard dependency.
There is a possibility though to split off strategies into separate packages so that the basic strategies are included but the ones with auto injection (which requires the league/container) and others can be separate packages. Like tactician-container.

@hannesvdvreken I actually toyed with this idea but ended up deciding against it, mainly because of the reasons above whereas if that was to happen I'm not sure of the use for this package as it stands. Those strategies as an extra package could just as easily be written as an extension to FastRoute, which is then a mirror of what league/route is. It's a never ending circle so I decided on the make it easier by default without going overboard with the option to change anything you want to.

@thebigb although league/container is included, the code itself is only dependent on container-interop interfaces so that they can be swapped out, the argument here is whether league/container should be included for use by default. The reason that it is, and the reason that it will stay this way is to make the package easier to use for anybody who just wants to install and go.

Docs....


Structure hashing settings neatly

This includes:

Number of hashes per field
Bloom filter length
Number of XOR folds

This was done with the schema.


Generate swifty typealias

It looks like we can generate typealias like this:
typealias TWBitcoinSigningInput = TW_Bitcoin_Proto_SigningInput
typealias TWBitcoinUnspentTransaction = TW_Bitcoin_Proto_UnspentTransaction
typealias TWBitcoinTransactionPlan = TW_Bitcoin_Proto_TransactionPlan
typealias TWBitcoinOutPoint = TW_Bitcoin_Proto_OutPoint

We can remove TW prefix once we remove TrustCore's Bitcoin implementation

@hewigovens @alejandro-isaza should we still need these aliases? or already implemented?

Yes, we should still do this. Right now it's a manual process.


Get the quality and timestamp

Hello.
Is it possible to get quality and a timestamp?

or other arbitrary fields

Do you mean while monitoring?

reading and monitoring, and set on write

I'm not sure I understand what you're asking.

I want to read the value as well as its quality and time stamp of when it was installed. So I want to set the quality of the installation of values.

I'm sorry. When what was installed? The OPC server?

I want to read value, quality and time stamp. And I want to set the quality on value write.

I understand now. I'll look into it

This is definitely possible. These results are gotten but basically discarded.

I've already done this, here is my DA and UA code...
`public enum TagQualities
{
[DescriptionAttribute("None")]
None,
[DescriptionAttribute("Bad")]
Bad,
[DescriptionAttribute("Good")]
Good
}
public Practicon.OPC.Common.TagQualities TagQuality(string tagName)
{
    var item = new OpcDa.Item { ItemName = tagName };
    if (Status == OpcStatus.NotConnected)
    {
        throw new OpcException("Server not connected. Cannot read tag.");
    }
    var result = _server.Read(new[] { item })[0];
   if (result.Quality == OpcDa.Quality.Bad)
   {
       return TagQualities.Bad;
   }
   else if (result.Quality == OpcDa.Quality.Good)
   {
       return TagQualities.Good;
   }
   return TagQualities.None;
}`

//     /// <summary>     /// Tag's TimeStamp     /// </summary>     /// <param name="tagName"></param>     /// <returns></returns>     public DateTime TimeStamp(string tagName)     {         var item = new OpcDa.Item { ItemName = tagName };         if (Status == OpcStatus.NotConnected)         {             throw new OpcException("Server not connected. Cannot read tag.");         }         var result = _server.Read(new[] { item })[0];         return result.Timestamp;     }
UA Code
`/
///
/// Tag's TimeStamp
///
///
///
public DateTime TimeStamp(string tagName)
{
var nodesToRead = BuildReadValueIdCollection(tagName, Attributes.Value);
        DataValueCollection results;
        DiagnosticInfoCollection diag;
        _session.Read(
            requestHeader: null,
            maxAge: 0,
            timestampsToReturn: TimestampsToReturn.Server,
            nodesToRead: nodesToRead,
            results: out results,
            diagnosticInfos: out diag);
        var val = results[0];

        return val.ServerTimestamp;
    }

 public Practicon.OPC.Common.TagQualities TagQuality(string tagName)
{
        var nodesToRead = BuildReadValueIdCollection(tagName, Attributes.Value);

        DataValueCollection results;
        DiagnosticInfoCollection diag;
        _session.Read(
            requestHeader: null,
            maxAge: 0,
            timestampsToReturn: TimestampsToReturn.Server,
            nodesToRead: nodesToRead,
            results: out results,
            diagnosticInfos: out diag);
        var val = results[0];

        if (StatusCode.IsGood(val.StatusCode))
        {
            return TagQualities.Good;
        }
        else if (StatusCode.IsBad(val.StatusCode))
        {
            return TagQualities.Bad;
        }
        return TagQualities.None

}`
I've haveto use my own enumeration for Quality since I need  a  common result type to pass to COM clients...

That's very helpful. We'll also want to add it to the Node classes so we can retrieve these when reading tag's value. That way, it could optionally come back in the monitor callback in case you want to monitor it yourself

Hi guys, great work with the library. This specific point is also very important for me, I can do the changes in a branch, and then we can merge it, would that be fine by you?
Thanks

@bajcmartinez, of course it would!

I have the changes ready, but can't push a new branch, can you give me access or you prefer me to do a fork? Thanks

I'd prefer you to do a fork and then make a pull request. Thanks!


add missing compile step to 'running-rntesterexamples' documentation

It seems that packages rndom-redbox and rndom-switch need to be compiled in order to run the Examples.

Thanks for this!


rds: changing to factory instantiation

Risk Level: low
Testing: updated tests
Docs Changes: n/a
Release Notes: n/a
https://github.com/envoyproxy/envoy-mobile/issues/176

/retest


uop global buf number tracking try 2 [pr]

really don't like that llama unjitted is 30% slower

so if I just keep track of state, it's a little bit slower, where is the 30% coming from?

meh, global state is expensive


Can't build site with 'hugo' command

A fresh install following the README, fails to work when the hugo command it run. I'm really not sure where the issue is, I'm guessing during the purge as hugo server works fine.
Using Hugo v0.67.0/extended darwin/amd64
Note: I changed the path names in the following copy and paste to protect my privacy
Building sites … ERROR 2020/09/17 15:43:30 render of "taxonomyTerm" failed: execute of template failed: template: _default/list.html:3:7: executing "_default/list.html" at <partial "head.html" .>: error calling partial: "test-tailwind/themes/new-theme-name/layouts/partials/head.html:22:55": execute of template failed: template: partials/head.html:22:55: executing "partials/head.html" at <resources>: can't evaluate field PostProcess in type interface {}

ERROR 2020/09/17 15:43:30 render of "taxonomyTerm" failed: execute of template failed: template: _default/list.html:3:7: executing "_default/list.html" at <partial "head.html" .>: error calling partial: "test-tailwind/themes/new-theme-name/layouts/partials/head.html:22:55": execute of template failed: template: partials/head.html:22:55: executing "partials/head.html" at <resources>: can't evaluate field PostProcess in type interface {}
Total in 20 ms

Error: Error building site: failed to render pages: render of "home" failed: execute of template failed: template: index.html:3:7: executing "index.html" at <partial "head.html" .>: error calling partial: "test-tailwind/themes/new-theme-name/layouts/partials/head.html:22:55": execute of template failed: template: partials/head.html:22:55: executing "partials/head.html" at <resources>: can't evaluate field PostProcess in type interface {}

Looks like my Hugo version was too far behind. I've upgraded, and the problem was solved!


[release/1.2] backport: Disable criu tests in Travis CI

Temporarily disable criu tests until the 5.0.0 CI kernel issue is
resolved. Also update criu to v3.13
Cherry-picked: #3898 for release/1.2
Signed-off-by: Phil Estes<EMAIL_ADDRESS>

LGTM

Codecov Report

Merging #3909 into release/1.2 will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           release/1.2    #3909   +/-   ##
============================================
  Coverage        44.19%   44.19%
============================================
  Files              100      100
  Lines            10847    10847
============================================
  Hits              4794     4794
  Misses            5313     5313
  Partials           740      740

Flag
Coverage Δ

#linux
47.87% <ø> (ø)
:arrow_up:

#windows
41% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Bumping drivers from 1.7 to 1.8

Bumping drivers from 1.7 to 1.8

Closing in favor of #201 because forks are hard


update spidernet v0.3.0 release note

please refer to : https://gitlab.daocloud.cn/ndx/engineering/network/spidernet/-/tree/release-v0.3.0/release-notes/notes/v0.3.0

Noted. Added todo.


duel frag limits are too high

let's set it to 5 for w/ weapons and 3 for w/o.

updated frag limits on duels
https://github.com/LacledesLAN/gamesvr-jk2outcast-freeplay/commit/9217f47823f66e1d7053850ef44f749368e0b039

updated frag limits on duels
https://github.com/LacledesLAN/gamesvr-jk2outcast-freeplay/commit/9217f47823f66e1d7053850ef44f749368e0b039

issue fixed as of bc66de698e627d24e51821d8a456eec3a74ee716


Creating a script tag returns obfuscated error

creating a script tag via the Shopify node api returns a 400 but the error does not explain why.
const data = await ctx.state.client.post({
            path: 'script_tags',
            body: {
                "script_tag": {
                    "event": "onload",
                    "src": "https://google.com"
                }
            },
            type: DataType.JSON,
        });

returns:
Received an error response (400 Bad Request) from Shopify: [object Object]. If you report this error, please include this id: 764305ab-f162-491c-9ab9-7ce93a835f09

Full code:
import Shopify, { DataType } from "@shopify/shopify-api";
const Router = require("koa-router")
const router = new Router({ prefix: "/scriptTags" })

router.use(async (ctx, next) => {
    const session = await Shopify.Utils.loadCurrentSession(ctx.req, ctx.res)

    if (!session?.shop || !session?.accessToken) {
        throw new Error("session could not be loaded. unable to create rest client")
    }

    ctx.state.client = new Shopify.Clients.Rest(session.shop, session.accessToken);
    await next()
})

router.post('/', async ctx => {
    try {
        console.log("ctx.state.client", ctx.state.client) // logs Rest Client as expected
        const data = await ctx.state.client.post({
            path: 'script_tags',
            body: {
                "script_tag": {
                    "event": "onload",
                    "src": "https://google.com"
                }
            },
            type: DataType.JSON,
        });
        console.log("data", data)
        ctx.body = data
    } catch (e) {
        console.log("error", e, e.message, e.code);
    }
})

I figured it out after wasted many, many hours. The Shopify documentation is wrong and lists body as a key to the restclient#post params when it should be data

So much frustration 😭


ErrorCode: authentication_ui_failed

I went through the steps here and received the following Error:
Error Acquiring Token:
MSAL.UAP.<IP_ADDRESS>.MsalClientException:
ErrorCode: authentication_ui_failed
Microsoft.Identity.Client.MsalClientException: Web Authentication Broker (WAB) authentication failed. To collect WAB logs, please follow https://aka.ms/msal-net-wab-logs ---> System.Exception: The process terminated unexpectedly. (Exception from HRESULT: 0x8007042B)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
It fails at MainPage.xaml.cs = Line 65
IEnumerable<IAccount> accounts = await PublicClientApp.GetAccountsAsync().ConfigureAwait(false);
No accounts are found.
I attempted this also by creating an App Registration in Azure and then letting the tool generate the code for me.  I do like how you template in the ClientId into the code.  However, I still get the same error when I execute.
Has something changed since this was published?

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

ID: 5cceddcd-1431-ab61-df75-7ccf74a84fba
Version Independent ID: d5833a30-c43e-8048-63b3-d94d26410a3b
Content: Microsoft identity platform Windows UWP quickstart
Content Source: articles/active-directory/develop/quickstart-v2-uwp.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

BTW, I've done the WPF version of this and it worked perfect.

@keithnicholson Thank you for making us aware of this issue. We are investigating and will update this issue with the resolution steps when we understand the root issue.

@keithnicholson : do you mind opening an issue on the MSAL.NET repo?
https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/issues/new?assignees=&labels=&template=bug_report.md&title=[Bug]+

We fixed the UWP Quickstart last week in all clouds. It should get live soon
#please-close
cc: @mmacy


Cannot run adonis--help after i install

I use a mac and all i see after is this;
-bash: adonis: command not found
I am running v10.16.0 of node and v6.9.0

I am new to adonis so I dont really know what is going on

Opanins-MBP:~ oooapss$ npm i -g @adonisjs/cli
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade to kleur@3 or migrate to 'ansi-colors' if you prefer the old syntax. Visit https://github.com/lukeed/kleur/releases/tag/v3.0.0\ for migration path(s).
/Users/oooapss/.npm-global/bin/adonis -><EMAIL_ADDRESS>
<EMAIL_ADDRESS>updated 1 package in 46.494s

after all this still same error


Create documentation for module- Payments

Generate the DFD diagram for the module
Identify and remove unused code
Generate coverage report
Add module level readme

Hi i'd like to work on this issue.

Hey @SaurabhGarg6174 , please feel free to work on this issue. Assigned to you.

@SaurabhGarg6174 Do we have any updates on this?


[Suggestion] Disable or improve stickers

Is your feature request related to a problem? Please describe.
I use a different plugin, FreeStickers, to send links to stickers. EmoteReplacer currently sends a blurry rescaled image. With both plugins enabled, the sticker is sent as a link and as a blurry image.

Describe the solution you'd like
Provide an option to disable the sticker sending functionality and/or improve the resolution of the sticker image sent. The sticker used in this example appears to have a max resolution of 128x72 px. It might be necessary to check the size of a sticker before attempting to rescale it.

Go into settings and select "Scale down largest side" under "Resize method"

That's much better, but it's still upscaling the image resulting in some loss of quality.

Stickers will scale using url where possible in 2.1.5, if this doesn't return a 160px image then scaling will still need to be done by the plugin to match the standard 160px size for stickers.


Console error adding -Wconf:any:error

steps
ThisBuild / scalaVersion := "2.13.3"
Compile / console / scalacOptions ++= Seq("-Wconf:any:error")

to reproduce:

enter the sbt console
enter 42 // show

problem
I'm getting the following error
scala> 42 // show
package $line3 {
  sealed class $read extends _root_.scala.Serializable {
    def <init>() = {
      super.<init>;
      ()
    };
    sealed class $iw extends _root_.java.io.Serializable {
      def <init>() = {
        super.<init>;
        ()
      };
      val res1 = 42
    };
    val $iw = new $iw.<init>
  };
  object $read extends scala.AnyRef {
    def <init>() = {
      super.<init>;
      ()
    };
    val INSTANCE = new $read.<init>
  }
}

package $line3 {
object $eval {
  lazy val $result = $line3.$read.INSTANCE.$iw.res1
  lazy val $print: _root_.java.lang.String =  {
    $line3.$read.INSTANCE.$iw

""  + "val res1: Int" + " = " + _root_.scala.runtime.ScalaRunTime.replStringOf($line3.$read.INSTANCE.$iw.res1, 1000)

  }
}}

           $line3.$read.INSTANCE.$iw
                                 ^
<synthetic>:6: error: a pure expression does nothing in statement position; multiline expressions might require enclosing parentheses

(To diagnose errors in synthetic code, try adding `// show` to the end of your input.)

expectation
no errors. I would expect something like
scala> 42 // show
package $line2 {
  sealed class $read extends _root_.scala.Serializable {
    def <init>() = {
      super.<init>;
      ()
    };
    sealed class $iw extends _root_.java.io.Serializable {
      def <init>() = {
        super.<init>;
        ()
      };
      val res0 = 42
    };
    val $iw = new $iw.<init>
  };
  object $read extends scala.AnyRef {
    def <init>() = {
      super.<init>;
      ()
    };
    val INSTANCE = new $read.<init>
  }
}

package $line2 {
object $eval {
  lazy val $result = $line2.$read.INSTANCE.$iw.res0
  lazy val $print: _root_.java.lang.String =  {
    $line2.$read.INSTANCE.$iw

""  + "val res0: Int" + " = " + _root_.scala.runtime.ScalaRunTime.replStringOf($line2.$read.INSTANCE.$iw.res0, 1000)

  }
}}
val res0: Int = 42

notes
scala: 2.13.3
sbt: 1.3.8
ref: https://github.com/laserdisc-io/laserdisc/pull/315

@barambani Thanks for the detailed bug report. I was able to reproduce the error on my machine as well.
However, I disagree that this is an sbt problem. sbt basically executes Scala's REPL program, and the error in question seems to be coming from 2006 - https://github.com/scala/scala/commit/01443e42ed009c7125fe5b5c07ec20ff5eadbd17#diff-6eef86d28757da64bbaee0b568952750R332.
I suggest you report it to scala/bug.

Here's my PR for this - https://github.com/scala/scala/pull/9161

Thanks for the clarification @eed3si9n and for the fix. If this doesn't make sense here I will close it then.


Bug: Phone icon is missing from the Home Page drives widget

Description
I've noticed an issue with my phone's connectivity. While the icon in the sidebar under "Drives" displays correctly when I connect my phone, the same icon doesn't appear on the home page.
Steps To Reproduce

Connect a phone

Requirements
Ensure that when a phone is connected, the icon is synchronized to appear consistently, both in the sidebar under "Drives" and on the home page.
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.3155
Log File

Thank you for your feedback. I can repro as well.

Is this a new issue or was it present in older versions as well?

@0x5bfa @Lukiluc29 is this fixed in main?


Fix simple table printing

makes it not die of recursion
also fixes printing tables generally

this does not conflict with #73


added Code With Chris to list of participants and to apps table

Hey John, great initiative. I'd love to participate!

Fantastic, glad to have you on board @codewithchris 🎉


"left_subtract" not defined in SULOV

Hello,
When I execute
outputs = featurewiz.featurewiz(features_train.join(y_train), "label", corr_limit=0.7, verbose=1)
I receive the error message
#######################################################################################
#####  Searching for Uncorrelated List Of Variables (SULOV) in 446 features ############
#######################################################################################
    there are no null values in dataset...
    SULOV Method crashing due to name 'left_subtract' is not defined
    SULOV method is erroring. Continuing ...
Time taken for SULOV method = 2 seconds
    Adding 0 categorical variables to reduced numeric variables  of 446
Final list of selected vars after SULOV = 446

Also, when I import the left_subtract function, it still doesnt work.
from featurewiz.featurewiz import left_subtract
What is the issue here?

I have the same issue!

Sorry @hempelc @Kayne88 :
I made a mistake and left out left_subtract in a couple of files when I made a recent modification to the code base. It is now corrected in the latest version 0.1.43
Please upgrade via
pip install featurewiz --upgrade --ignore-installed --no-deps

Thanks
AutoViML


Remove AlignTrailingComments format option

This option existing in the config seems to cause clang-format 16 to error with
.clang-format:9:24: error: not a mapping
AlignTrailingComments: True
                       ^~~~
Error reading .clang-format: Invalid argument

Relevant comment: https://github.com/stblr/mkw-sp/issues/829#issuecomment-1581391156

Sure, but the C++ VScode extension seems to be bundling clang-format 16, and the option doesn't change how formatting on SP is done now. (see: I haven't had to touch CI and it's passing).

If no BasedOnStyle is selected, clang-format defaults to the LLVM style. The LLVM style has AlignTrailingComments as "true"; therefore, we can simply delete this line and avoid all of this headache.


lots of "TODO: use x from MacCompileStuff" comments

Currently SourcesAndScripts/Scripts/functions contains a lot of checks for things being installed in /usr/local, and suggests to use Homebrew to install things that are missing. However, users might not use Homebrew as their package manager (I use MacPorts for instance, and then there's also Fink), and even if they do use Homebrew, it could be installed to a non-default prefix. And in any case, MacCompileStuff provides all the packages being requested itself, so it should just use its own packages when possible (as the TODO comments say) instead of depending on an external package manager.

Yes, that applies to master branch, but it should be solved in universal_libs_support branch which is default for 1.15 and future 1.16 https://github.com/hrubymar10/MacCompileStuff/blob/universal_libs_support/SourcesAndScripts/Scripts/env
Is that solution to this issue?

ok switching to the universal_libs_support branch gets me farther, but then leads to a build failure when freetype attempts to link against non-universal libs installed in /opt/local by MacPorts:
libtool: link: clang -dynamiclib  -o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/libfreetype.6.dylib  /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftsystem.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftdebug.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftinit.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftbase.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftbbox.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftbdf.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftbitmap.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftcid.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftfstype.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftgasp.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftglyph.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftgxval.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftmm.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftotval.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftpatent.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftpfr.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftstroke.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftsynth.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/fttype1.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftwinfnt.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/truetype.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/type1.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/cff.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/type1cid.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/pfr.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/type42.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/winfnt.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/pcf.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/bdf.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/sfnt.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/autofit.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/pshinter.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/raster.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/smooth.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftcache.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftgzip.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftlzw.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/ftbzip2.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/psaux.o /Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/psnames.o   -L/Users/ericgallager/Documents/GitHub/MacCompileStuff/lib -L/opt/local/lib -lz -lbz2 -lpng  -arch arm64 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk -mmacosx-version-min=10.11   -install_name  /usr/local/lib/libfreetype.6.dylib -compatibility_version 24 -current_version 24.4 -Wl,-single_module -Wl,-exported_symbols_list,/Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/.libs/libfreetype-symbols.expsym
ld: warning: ignoring file /opt/local/lib/libz.dylib, building for macOS-arm64 but attempting to link with file built for macOS-x86_64
ld: warning: ignoring file /opt/local/lib/libbz2.dylib, building for macOS-arm64 but attempting to link with file built for macOS-x86_64
ld: in '/opt/local/lib/libunwind.dylib', building for macOS-arm64 but attempting to link with file built for macOS-x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [/Users/ericgallager/Documents/GitHub/MacCompileStuff/SourcesAndScripts/Sources/freetype-2.10.4-arm64/objs/libfreetype.la] Error 1

I'll have to see if I can edit the script to get it to ignore my MacPorts prefix...

universal_libs_support can be build only on native Apple Silicon mac. It was necessary at time of creating that branch, because some libraries did not fully support cross-compiling arm targets yet. I expect that situation is better now, but I don't think that everything is solved. The most problems IIRC was with GNOME libraries built using meson+ninja.

universal_libs_support can be build only on native Apple Silicon mac. It was necessary at time of creating that branch, because some libraries did not fully support cross-compiling arm targets yet. I expect that situation is better now, but I don't think that everything is solved.

Could you check up on said situation again please, @hrubymar10?

I will close this issue in favor of #3 . There is updated info regarding cross compilation.
TL;DR: I won't invest time in it. I don't have enough time for it and also I don't find it important.