gcore provider not working

apiVersion: se.quencer.io/v1alpha1
kind: DNSIntegration
metadata:
  labels:
    argocd.argoproj.io/instance: k8s-blue-cc-phonebook
  name: gcore
spec:
  provider:
    name: gcore
  secretRef:
    keys:
      - key: GCORE_PERMANENT_API_TOKEN
        name: GCORE_API_TOKEN
    name: phonebook-secrets
  zones:
    - myzone.com

Name:         gcore
Namespace:
Labels:       argocd.argoproj.io/instance=k8s-blue-cc-phonebook
Annotations:  <none>
API Version:  se.quencer.io/v1alpha1
Kind:         DNSIntegration
Metadata:
  Creation Timestamp:  2024-11-01T20:06:44Z
  Finalizers:
    phonebook.se.quencer.io/deployment
  Generation:        1
  Resource Version:  90979843
  UID:               b0c39020-951e-448c-8bbd-df03550567e0
Spec:
  Provider:
    Name:  gcore
  Secret Ref:
    Keys:
      Key:   GCORE_PERMANENT_API_TOKEN
      Name:  GCORE_API_TOKEN
    Name:    phonebook-secrets
  Zones:
    wxs.ro
Status:
  Conditions:
    Last Transition Time:  2024-11-01T20:06:44Z
    Reason:                Deployment.apps "provider-gcore" is invalid: spec.template.spec.containers[0].image: Required value
    Status:                Error
    Type:                  Deployment
Events:                    <none>

Sorry about that, the release process is very manual and error prone. I have rebuilt and reshipped 0.3.7. Usually, helm chart versions needs to increment but I didn't do it.
Can you try again? You might need to remove phonebook from your helm repo or at the very least force an update.

No worries, but i tried everything that I could think off, I am deploying with argo, I've tried hard refresh, removing the app, etc, nothing worked, same error, I suspect this is still present.
Can you please reopen it and maybe bump the chart version so we can exclude the caching of the old version?
Thank you!

@WladyX Before I make changes to the helm chart can you test this DNSIntegration for me first?
apiVersion: se.quencer.io/v1alpha1
kind: DNSIntegration
metadata:
  labels:
    argocd.argoproj.io/instance: k8s-blue-cc-phonebook
  name: gcore
spec:
  provider:
    name: gcore
    image: "ghcr.io/pier-oliviert/providers-gcore:v0.3.7"
  secretRef:
    keys:
      - key: GCORE_PERMANENT_API_TOKEN
        name: GCORE_API_TOKEN
    name: phonebook-secrets
  zones:
    - myzone.com

You can specify images directly and I want to make sure the image loads OK for you. One thing I realized is that all images are built for x64 platform so I want to make sure the image is actually running in your cluster.

still does not work and i don't think it's related to x64.
❯ kd dnsintegrations.se.quencer.io gcore
Name:         gcore
Namespace:
Labels:       argocd.argoproj.io/instance=k8s-blue-cc-phonebook
Annotations:  <none>
API Version:  se.quencer.io/v1alpha1
Kind:         DNSIntegration
Metadata:
  Creation Timestamp:  2024-11-02T12:09:00Z
  Finalizers:
    phonebook.se.quencer.io/deployment
  Generation:        2
  Resource Version:  91942169
  UID:               a91c64b1-7d34-4a1b-83bc-fb9a00380131
Spec:
  Provider:
    Image:  ghcr.io/pier-oliviert/providers-gcore:v0.3.7
    Name:   gcore
  Secret Ref:
    Keys:
      Key:   GCORE_PERMANENT_API_TOKEN
      Name:  GCORE_API_TOKEN
    Name:    phonebook-secrets
  Zones:
    myzone.com
Status:
  Conditions:
    Last Transition Time:  2024-11-02T12:09:22Z
    Reason:                Deployment.apps "provider-gcore" is invalid: spec.template.spec.containers[0].image: Required value
    Status:                Error
    Type:                  Deployment
Events:                    <none>
❯ kg dnsintegrations.se.quencer.io gcore -oyaml|kneat
apiVersion: se.quencer.io/v1alpha1
kind: DNSIntegration
metadata:
  labels:
    argocd.argoproj.io/instance: k8s-blue-cc-phonebook
  name: gcore
spec:
  provider:
    image: ghcr.io/pier-oliviert/providers-gcore:v0.3.7
    name: gcore
  secretRef:
    keys:
      - key: GCORE_PERMANENT_API_TOKEN
        name: GCORE_API_TOKEN
    name: phonebook-secrets
  zones:
    - myzone.com

scratch that, i removed the dnsintegration gcore and redeployed it and the pod started, will do some more tests and come back, thank you!

@WladyX I was going crazy over here, looking at the code I couldn't understand what was going on and was about to create a discord server to discuss with you.
Glad you got it working!

i just tried deleting the dnsintegration and testing without image spec, still does not work, so pls bump the chart, so I can test it without image in dnsintegration.
the record was created ok on gcore when I tried with the image specified, but I would have expected an error from desec, since it should try to create the record on both, but that's another story, I can opened another issue if you like.
thank you!


Remove HTTP "Connection" header when forwarding requests

Scope & Purpose
Remove HTTP "Connection" header when forwarding requests in the cluster from one coordinator to another, and let the internal network layer handle closing of connections and keep-alive.

[x] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Backports:

[x] Backports required for: 3.6

Testing & Verification

[x] This change is a trivial rework / code cleanup without any test coverage.
[ ] The behavior in this PR was manually tested
[ ] This change is already covered by existing tests, such as (please describe tests).
[x] This PR adds tests that were used to verify all some changes:

[x] Added new integration tests (e.g. in load-balancing)

Link to Jenkins PR run:
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13351/

Tests blue

confirmed to fix https://github.com/arangodb/arangojs/issues/699


Filtering: :typealias

I noticed that https://github.com/MichaelHatherly/Lexicon.jl/blob/master/src/filtering.jl#L13 says:

* `categories` : categories to include in the result; can include any
  of [:module, :function, :method, :type, :macro, :global, :comment];

But at the Sims.jl - build.jl there is an additional ** :typealias**.
categories = [:comment, :module, :function, :method, :type, :typealias, :macro, :global])

Maybe I miss something / or something changed - otherwise I think the docs are incorrect.

Aren't this all valid categories: https://github.com/MichaelHatherly/Docile.jl/blob/master/src/macros.jl#L6
object_category(ex) =
    ismethod(ex) ? :method    :
    ismacro(ex)  ? :macro     :
    istype(ex)   ? :type      :
    isalias(ex)  ? :typealias :
    isglobal(ex) ? :global    :
    issymbol(ex) ? :symbol    :
    istuple(ex)  ? :tuple     :
    isvcat(ex)   ? :vcat      :
    isvect(ex)   ? :vect      :
    error("@doc: cannot document object:\n$(ex)")

Maybe I miss something / or something changed - otherwise I think the docs are incorrect.

Yeah, seems the docs just weren't updated along with the addition of typealias.

Aren't this all valid categories:

The last four, :symbol to :vect, allow for the special syntax for documenting functions/modules or groups or functions and methods:
" a function " # `:symbol`
foobar

" a function prior to isn't definition " # `:symbol`
:baz

" a method with argument types Int and Float64 " # `:tuple`
(foobar, Int, Float64)

" two completely different functions " # `:vcat`/`:vect` depending on version 0.3/0.4
[foobar, baz]

The last four, :symbol to :vect, allow for the special syntax for documenting functions/modules or groups or functions and methods:

this types can not be filtered? or are they translated internally to one of the others?

They're internal details and get expanded into multiple docstring entries. See here and here.


WebDriver (WebDriverIO 5) scrollTo does not work due to deprecated API

What are you trying to achieve?
Run test with scrollTo on safari 12 and firefox 65 with WebDriver helper (WebDriverIO 5)
What do you get instead?

Provide console output if related. Use --verbose mode for more details.

TypeError: this.browser.getElementLocation is not a function
      at WebDriver.scrollTo (node_modules/codeceptjs/lib/helper/WebDriver.js:1248:43)
      at <anonymous>
      at process._tickDomainCallback (internal/process/next_tick.js:229:7)

Details

CodeceptJS version: 2.0.4
NodeJS Version: 8.14.3
Operating System: MacOS High Sierra
WebDriver

It's all about WebDriverIO 5 breaking changes:
const location = await this.browser.getElementLocation(elementId);
getElementLocation was deprecated. element.getLocation should be used now.
https://webdriver.io/docs/api/element/getLocation.html
I have a variant to fix it. But will do it in a few days, if someone will not do it earlier.

I think it is the same
TypeError: this.browser.getLocation is not a function
      at WebDriver.scrollTo (node_modules/codeceptjs/lib/helper/WebDriver.js:1248:43)
      at process._tickCallback (internal/process/next_tick.js:68:7)

@LukoyanovE  If you want, I can create a PR for this?

@PeterNgTr ,

if someone will not do it earlier.

If you want or if you need this as soon as possible, do.
I can start to fix it on monday, unless anybody fix.

Not huge changes, I could make it. https://github.com/Codeception/CodeceptJS/pull/1519

Fixed in #1519


Support import remapping

Add remapping to CompilerOptions.
http://solidity.readthedocs.io/en/develop/using-the-compiler.html?highlight=remapping

remappings field is added to CompilerOptions. Now we need a way to specify remappings via settings.

solidity.compilerOptions.remappings property is added in c557679c53014fa94c84736ec4474c36bab113b2


"bosh deploy" update of VMs blocked by "bosh ssh" - display warning to user?

One behavior I've seen before, and again just now - I run bosh deploy to update VMs (e.g. resize them) and the output blocks at Updating instance <name>.... Eventually you become suspicious that nothing is happening. Somewhere in your terminal tabs you discover you have a bosh ssh session into one of the VMs. You cancel the session and the bosh deploy can finally delete the vm.
Could we add something to the bosh deploy output which states that it is not going to do the update until the bosh ssh sessions are stopped, so you know to go find them and kill them?

that's interesting that it would block the deploy. is it just plain ssh or were you running something on the system?

a) not sure why I raised ticket against docs; b) I’ll try to reproduce

From: Dmitriy Kalinin<EMAIL_ADDRESS>Sent: Tuesday, April 24, 2018 4:55:45 PM
To: cloudfoundry/docs-bosh
Cc: Dr Nic Williams; Author
Subject: Re: [cloudfoundry/docs-bosh] "bosh deploy" update of VMs blocked by "bosh ssh" - display warning to user? (#501)
that's interesting that it would block the deploy. is it just plain ssh or were you running something on the system?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/cloudfoundry/docs-bosh/issues/501#issuecomment-384116503, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAAAbNA_pWiivguZ33WITVBYEQSk74mUks5tr7uBgaJpZM4Te2Xb.


PostgreSQL plugin support?

Is there any way to specify PostgreSQL plugins?
For example, one of my work projects requires TimescaleDB.
Thanks!

{ pkgs, ... }:

{
  postgres.enable = true;
  # see https://github.com/NixOS/nixpkgs/blob/master/pkgs/servers/sql/postgresql/packages.nix for full list
  postgres.package = pkgs.postgresql_13.withPackages (p: [ p.pg_cron p.plpgsql_check p.pg_partman ]);
}

This needs documentation in the same style as https://devenv.sh/reference/options/#languagesphppackage


Unable to upload files more than 65536 bytes

Hello,
We've started discussion here https://github.com/twitter/CocoaSPDY/pull/85, but there was no answer so I decided to open an issue.
There is a function:
(void)didReadWindowUpdateFrame:(SPDYWindowUpdateFrame )windowUpdateFrame frameDecoder:(SPDYFrameDecoder *)frameDecoder
{
/*

SPDY WINDOW_UPDATE frame processing requirements: *
Receivers of a WINDOW_UPDATE that cause the window size to exceed 2^31
must send a RST_STREAM with the status code FLOW_CONTROL_ERROR. *
Sender should ignore all WINDOW_UPDATE frames associated with a stream
after sending the last frame for the stream. */
SPDYStreamId streamId = windowUpdateFrame.streamId;
SPDY_DEBUG(@"received WINDOW_UPDATE.%u (+%lu)", streamId, (unsigned long)windowUpdateFrame.deltaWindowSize);

if (streamId == kSPDYSessionStreamId) {
// Check for numerical overflow
if (_sessionSendWindowSize > INT32_MAX - windowUpdateFrame.deltaWindowSize) {
[self _closeWithStatus:SPDY_SESSION_PROTOCOL_ERROR];
return;
}

_sessionSendWindowSize += windowUpdateFrame.deltaWindowSize;
for (SPDYStream *stream in _activeStreams) {
    [self _sendData:stream];
    if (_sessionSendWindowSize == 0) break;
}

return;
}

// Ignore frames for non-existent or half-closed streams
SPDYStream *stream = _activeStreams[streamId];
if (!stream || stream.localSideClosed) {
return;
}

// Check for numerical overflow
if (stream.sendWindowSize > INT32_MAX - windowUpdateFrame.deltaWindowSize) {
[self _sendRstStream:SPDY_STREAM_FLOW_CONTROL_ERROR streamId:streamId];
return;
}

stream.sendWindowSize += windowUpdateFrame.deltaWindowSize;
[self _sendData:stream];
}

if streamId is eqal to kSPDYSessionStreamId then _sessionSendWindowSize should be increased, but if not, stream.sendWindowSize should be increased instead. So only one sendWindowSize will be increased, not the both.
In _sendData:(SPDYStream *)stream function both of sendWindowSize were decreased at the same time:
_sessionSendWindowSize -= bytesSent;
stream.sendWindowSize -= bytesSent;

and in case of bytesSent equals 65536 bytes, they both equal 0.
Upon next call to the _sendData this line will always return 0:
uint32_t sendWindowSize = MIN(_sessionSendWindowSize, stream.sendWindowSize);

because only one sendWindowSize was increased in didReadWindowUpdateFrame.
So, a file with size more than 65536 bytes will never send.

@chexov Did you find anything interesting?

@kgoodier thanks for waiting. I did dig it today a little bit and spdy server receives windowUpdates for the opened session.
I am wrong somewhere for sure here but bear with me. Here is debugging session of me as non iOS developer:

got jetty-spdy server running and make sure that have breakpoints set on settings+windowUpdate event to catch what did client sends me.

Got breakpoint triggered and cocoaspdy sends me settings with windowSize=10485760.
It starts from
[[SPDYSession alloc] initWithOrigin...]
which makes it to
SPDYSession.m:165 [self _sendWindowUpdate:deltaWindowSize streamId:kSPDYSessionStreamId]
which calculates deltaWindowSize from _sessionReceiveWindowSize var.

At this point we have cocoaspdy with _sessionSendWindowSize=65536 and spdy-server with windowSize=10485760.. which looks like a bug to me...No?

I changed [SPDYSession _sendWindowUpdate] method to look like this (added one line
_sessionSendWindowSize =  deltaWindowSize; and It works for my usecase...
Could be dirty hack but I am not familiar with the lib so close and hoping this will give you enough info to explain what I am doing wrong...

Full implementation looks like this:
- (void)_sendWindowUpdate:(uint32_t)deltaWindowSize streamId:(SPDYStreamId)streamId
{
    SPDYWindowUpdateFrame *windowUpdateFrame = [[SPDYWindowUpdateFrame alloc] init];
    windowUpdateFrame.streamId = streamId;
    windowUpdateFrame.deltaWindowSize = deltaWindowSize;
    _sessionSendWindowSize =  deltaWindowSize; // ADDED by @chexov
    [_frameEncoder encodeWindowUpdateFrame:windowUpdateFrame];
    SPDY_DEBUG(@"sent WINDOW_UPDATE.%u (+%lu)", streamId, (unsigned long)deltaWindowSize);
}

P.S. If you see no error on cocoaspdy side please let me know so I can continue to dig... really interested at fixing this. thanks!

server set window size is 2^31-1, same question appear。


More modular interface?

Is it possible to have some sort of interface that's not lm-format-enforcer wrapping vLLM, rather, users can directly install both library and configure it using SamplingParams?
The pseudocode in my mind:
SamplingParams(
    logits_processors=[lmformatenforcer.JSONSchema(MyPydanticClass)],
    max_tokens=16,
)

Here's a similar proposal to Outlines: https://github.com/outlines-dev/outlines/issues/163#issuecomment-1820069503

the logits processor needs the tokenizer to prepare itself. To make an API like this possible, the logitsprocessor interface would have to be more than a Callable, but a class with some init(tokenizer) functionality. This is inconsistent with how other inference libraries open logits processing APIs, which is why I chose this approach with vLLM.
Its technically possible, but would require changes to vLLM's SamplingParams, and I actually think the current interface is the right one.


Message Sending Activity graph

For non-americans if you run the server the dates are stored in dd/mm/yyyy however the graph still tries to display it in mm/dd/yyyy. So as you can see below, test messages were sent on the 3rd, 5th, and 6th of July but it is showing as if the days represented different months.

Thanks for bringing this to my attention. The graph is trying to decode the time as mm/dd/yyyy even when handed local time. I'll have this fixed soon.

fixed in af1af2840e2eada1733ab9793d7a2796e76df3d6


An accessibility modifier cannot be used with a private identifier

Suggestion
i think ts should allow the modifier private on a private identifier.
for keep a kind of design correlation in a project.
Actually change a public identifier to #private , lint error on all ts private modifier

Is there a way to ignore this error in tsconfigu ?
export class A  {
	private #foo= new foo();
}

i color all my accessibility modifier for fast nav, but private identifier lint error with the private accessibility modifier !

🔍 Search Terms
An accessibility modifier cannot be used with a private identifier

#private and private are mutually exclusive, like public and private. Typescript doesn't allow combining those either. It's confusing because they do nearly the same thing, but that's even more reason to keep them separate.

#private and private are mutually exclusive, like public and private. Typescript doesn't allow combining those either. It's confusing because they do nearly the same thing, but that's even more reason to keep them separate.
ho oki, i will take a look on doc to understand more !
thanks for your fast answer and your reason to keep this separate , i will close !

like public and private

Not quite, since they're not opposites.

they do nearly the same thing

Not that nearly, since private is enforced at compile time, and #private is enforced at runtime: https://stackoverflow.com/a/59641565/865175.


Transparent image not saved to the output folder

Why the transparent image is not saved to the output folder? Only the checkboard image is saved to the output folder?

I have the same issue, only the checkerboard image is saved and it discards the actual transparent image.

same

same

same

same

same, ongoing for over 2 weeks now!

temporary solution is to use the gradio temp folder which has the images from the generations. I don't know how often it clears but I ran trans for a couple hours and there are 400 images in the temp folder:
"C:\Users<youruser>\AppData\Local\Temp\gradio"

Same here.  Is there an option to just save the true-transparency PNG? I have no use for the fake checkered background image.  Thanks for the extension, is great!

i kjnow its not perfect, but read my last comment on how you can at least get the pngs with transparency:
temporary solution is to use the gradio temp folder which has the images from the generations. I don't know how often it clears but I ran trans for a couple hours and there are 400 images in the temp folder:
"C:\Users\AppData\Local\Temp\gradio"

Sure, I can find what I need there, but why the ordeal? I know this extension is on constant development and I would like to save transparency as a default since it's the extension main purpose. Thanks.


Dev to main sync

Developer Name: @sahsisunny
PR Number(s):-

#1630
#1625
#1660
#1649
#1663
#1666
#1653

Issue Ticket Number:-

#1629

Backend changes
Yes
Frontend Changes
Yes
No
Is Under Feature Flag
Yes
No
Database changes
Yes
No
Breaking changes (If your feature is breaking/missing something please mention pending tickets)
Yes
No
Deployment notes
Any special instructions for the deployment
Tested in staging
No
List of PRs going in this Sync.

#1630
#1625
#1660
#1649
#1663

Also attach their screenshots.

#1625

Staging Proff of PR #1653


Feat: Want to add reserved successfully page which open after click on reserve now.

Describe the feature
This feature will implement a "Reserved Successfully" page that users will be redirected to after they click the "Reserve Now" button. This page will serve as a confirmation of their reservation, providing essential details and enhancing the user experience.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a gssoc-24-extd contributor
[X] I want to work on this issue

@AbhijitMotekar99 not needed


release: 1.1.0

:robot: I have created a release beep boop
1.1.0 (2022-08-28)
Features

add startup (fca8abd)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/starudream/douyu-task/releases/tag/v1.1.0 :sunflower:


Provide documentation for Suitcase Upload

Software and hardware versions
ODK Suitcase 2.0.0
Problem description
Suitcase documentation doesn't cover uploading and resetting the app server: http://opendatakit-dev.cs.washington.edu/2_0_tools/release/214/suitcase
Furthermore the CLI help descriptions are quite brief and sometimes confusing. These should be updated and the documentation linked above should be updated to be more thorough and clear.
Also, I think the dataVersion CLI option should change to something like API version, or otherwise make it clear that this has nothing to do with the user's data.

This came up again: https://forum.odk-x.org/t/bulk-deleting-of-data/692

Hey @elmps2018,
I'd like to work on it.

@ADISKING1 just assigned it to you! Let me know any questions I can help with.

Thanks! I'll let you know if I'm stuck.

I have a question regarding the Upload option in Suitcase GUI.
How do we prepare a folder for Upload and what changes it makes to the server after we upload it?
Thank you.

@ADISKING1 The work done on https://github.com/odk-x/docs/pull/60 with the CLI shows some of the upload options. And this discussion on the Forum: https://forum.odk-x.org/t/using-suitcase-to-add-and-update-observations/301/6
describes some of the workflow. The basic idea is that you can update the data in tables on your server.

I have created a CSV and successfully used it to update, delete and add new entries via CLI but failed to do so using the GUI Upload option.
The Upload option in the GUI requires us to select a folder that contains target & assets folder. How do I setup those folders?

@ADISKING1 If you have the app designer set up, it should have the assets>csv folder where you could store uploads, is that what you mean? That's typically where one stores csvs that need to be imported to the tab/uploaded to the server. Or if there's something else if you can grab a screenshot of what you mean that would help!

@ADISKING1 there is a specific file structure that ODK-X uses. What @elmps2018 is referring to is inside config lives the assests folder. You upload the files in the correct file structure because that is where the ODK-X frameworks look for files. Here is a description of the file structure: https://docs.odk-x.org/config-file-structure/
NOTE: The files in the Application Designer zip follow the structure.

Thank you @wbrunette @elmps2018 for the clarification.
There is one more thing that's not clear to me. Can the GUI be used to UPDATE data?
The existing docs say that we can...

but here it is mentioned that it's possible only via the CLI

@ADISKING1 The Forum comment was from 2018 so I think the graphical interface can now upload as well (you should see that button now) but it needs to be documented more clearly.

I just checked a release older than that comment, but it still had the upload option.
There a possibility that GUI only supports UPLOAD and not UPDATE.

@ADISKING1 not sure, you can check the code and see what option is used. Code available here https://github.com/odk-x/suitcase

Just checked the code and found that GUI has a method call for UploadTask.java but no method calls for UpdateTask.java which is responsible for updating.
I believe that there is no option to update via GUI yet and I will write the docs accordingly.
Thank you @wbrunette, @elmps2018 for taking the time to address my doubts.


draggable alerts

Can the alerts be draggable?

Hello @j2shadow, thanks for the suggestion. this will be noted.

added this in v3.1.0


Import DOT format

https://en.wikipedia.org/wiki/DOT_(graph_description_language)

Write from scratch? Use an existing Java parser? Check license, no GPL,LGPL. We need example files.

You can use Maven to generate test files:
https://maven.apache.org/plugins/maven-dependency-plugin/usage.html#dependency:tree


Merged resources are stored in the wrong CR object and not applied to Pods

Describe the bug
With #175 we have added a feature that allows setting default resources. However, the resources that are merged with the template from the Schedule object and also specified in the job spec end up being stored in the Schedule CR, instead of being updated in the job CR.
this will remove the possibility to alter global default resources for cluster admins after the first reconciliation.
Additional context
See discussion here #260
Logs
The schedule gets updated with the following spec after reconcile:
apiVersion: backup.appuio.ch/v1alpha1
kind: Schedule
metadata:
  name: schedule-test
  namespace: default
spec:
...
  prune:
    resources:
      requests:
        cpu: 250m
    retention:
      keepDaily: 14
      keepLast: 5
    schedule: '*/1 * * * *'
  resourceRequirementsTemplate:
    requests:
      cpu: 250m

and prune:
apiVersion: backup.appuio.ch/v1alpha1
kind: Prune
metadata:
  name: prunejob-1609946760
  namespace: default
spec:
  resources:
    requests:
      cpu: 250m

Expected behavior

.spec.prune.resources should be empty in Schedule.
the Prune object should get all merged resources in the spec
K8up should schedule Pods according to the merged resources, but they aren't.

To Reproduce
Steps to reproduce the behavior:

Schedule Specs

---
apiVersion: backup.appuio.ch/v1alpha1
kind: Schedule
metadata:
  name: schedule-test
spec:
  resourceRequirementsTemplate:
    requests:
      cpu: "250m"
  prune:
    schedule: '*/1 * * * *'

Set global default memory limit

kubectl set env deployment/k8up-operator BACKUP_GLOBALMEMORY_LIMIT=76M

Environment (please complete the following information):

Image Version: v1.0.0-rc2
K8s Version: v1.18
K8s Distribution: KIND

Preventing the modification of the Schedule resource during reconciliation was already implemented in #260.

Preventing the modification of the Schedule resource during reconciliation was already implemented in #260.

I was just able to verify, based on this PR branch, that the schedule gets the memory from the global:
spec:
  backend:
    repoPasswordSecretRef:
      key: password
      name: backup-repo
    s3:
      accessKeyIDSecretRef:
        key: username
        name: backup-credentials
      bucket: k8up
      endpoint: 'http://<IP_ADDRESS>:9000'
      secretAccessKeySecretRef:
        key: password
        name: backup-credentials
  prune:
    resources:
      limits:
        cpu: 10m
    retention:
      keepDaily: 14
      keepLast: 5
    schedule: '*/1 * * * *'
  resourceRequirementsTemplate:
    requests:
      cpu: 250m
      memory: 76M

I was just able to verify, based on this PR branch, that the schedule gets the memory from the global:
spec:
  backend:
    repoPasswordSecretRef:
      key: password
      name: backup-repo
    s3:
      accessKeyIDSecretRef:
        key: username
        name: backup-credentials
      bucket: k8up
      endpoint: 'http://<IP_ADDRESS>:9000'
      secretAccessKeySecretRef:
        key: password
        name: backup-credentials
  prune:
    resources:
      limits:
        cpu: 10m
    retention:
      keepDaily: 14
      keepLast: 5
    schedule: '*/1 * * * *'
  resourceRequirementsTemplate:
    requests:
      cpu: 250m
      memory: 76M

ok, restart from clean:
make clean e2e-test
kubectl -n k8up-system set env deployment/k8up-operator BACKUP_GLOBALMEMORY_LIMIT=76M
k apply -f <the following file>

with this schedule:
---
apiVersion: backup.appuio.ch/v1alpha1
kind: Schedule
metadata:
  name: schedule-test
spec:
  resourceRequirementsTemplate:
    requests:
      cpu: "250m"
  backend:
    repoPasswordSecretRef:
      name: backup-repo
      key: password
    s3:
      endpoint: http://<IP_ADDRESS>:9000
      bucket: k8up
      accessKeyIDSecretRef:
        name: backup-credentials
        key: username
      secretAccessKeySecretRef:
        name: backup-credentials
        key: password
  prune:
    schedule: '*/1 * * * *'
    retention:
      keepLast: 5
      keepDaily: 14
    resources:
      limits:
        cpu: "500m"
        memory: 128M

I get the following properties in prune object:
spec:
  backend:
    repoPasswordSecretRef:
      key: password
      name: backup-repo
    s3:
      accessKeyIDSecretRef:
        key: username
        name: backup-credentials
      bucket: k8up
      endpoint: 'http://<IP_ADDRESS>:9000'
      secretAccessKeySecretRef:
        key: password
        name: backup-credentials
  resources:
    limits:
      cpu: 500m
      memory: 128M
    requests:
      cpu: 250m
      memory: 76M

The 76M should not appear at all if set via env var. The current commit doesn't fully resolve the bug. And somehow the Schedule object still gets it assigned in the resource Requirements template

ok, restart from clean:
make clean e2e-test
kubectl -n k8up-system set env deployment/k8up-operator BACKUP_GLOBALMEMORY_LIMIT=76M
k apply -f <the following file>

with this schedule:
---
apiVersion: backup.appuio.ch/v1alpha1
kind: Schedule
metadata:
  name: schedule-test
spec:
  resourceRequirementsTemplate:
    requests:
      cpu: "250m"
  backend:
    repoPasswordSecretRef:
      name: backup-repo
      key: password
    s3:
      endpoint: http://<IP_ADDRESS>:9000
      bucket: k8up
      accessKeyIDSecretRef:
        name: backup-credentials
        key: username
      secretAccessKeySecretRef:
        name: backup-credentials
        key: password
  prune:
    schedule: '*/1 * * * *'
    retention:
      keepLast: 5
      keepDaily: 14
    resources:
      limits:
        cpu: "500m"
        memory: 128M

I get the following properties in prune object:
spec:
  backend:
    repoPasswordSecretRef:
      key: password
      name: backup-repo
    s3:
      accessKeyIDSecretRef:
        key: username
        name: backup-credentials
      bucket: k8up
      endpoint: 'http://<IP_ADDRESS>:9000'
      secretAccessKeySecretRef:
        key: password
        name: backup-credentials
  resources:
    limits:
      cpu: 500m
      memory: 128M
    requests:
      cpu: 250m
      memory: 76M

The 76M should not appear at all if set via env var. The current commit doesn't fully resolve the bug. And somehow the Schedule object still gets it assigned in the resource Requirements template

Thank you @ccremer ! I'm now able to reproduce the issue. In the meantime, I believe #280 can be seen/merged as an independent fix.

Thank you @ccremer ! I'm now able to reproduce the issue. In the meantime, I believe #280 can be seen/merged as an independent fix.


Fix mouse scroll for IE

Fixes #3643

Thanks @lilleyse, looks like I'm the one who broke it to begin with.
Do a search for if (defined(document. there are actually 4 places this needs to be changed I think.

Thanks.


Disable the unique_subject requirement

Description
This allows more than one key with the same CN to exist in the CA's
database. Which is useful for renewal, as it's the only mode of renewal
supported by easy-rsa.
Issues Resolved

Not filed: Unable to create a new client with the same CN as a previous client. E.g. for renewal.

Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.

  Generated by :no_entry_sign: Danger

Tests are passing everywhere they do on master.

Rebased on master.


Blacklist Nicholas Mercadante from HAROLD

Time was spent, things were learned, and this has a definite benefit to house.
:ship: :shipit: :+1: :raised_hands: :tada:

I'd advise loading a blacklist from a configuration file instead of hard coding it.
Sent from my Android device with K-9 Mail. Please excuse my brevity.


Profiles are not displayed when initialising a LES with a technology matrix in CSV format

This LES does not show profiles.
For this LES, the tariffs are blank as well:

I see that the Buffers are blanked for this LES. It does have profiles attached or did you do that manually afterwards? Also stating that this issue is old, can this still be reproduced?

If I try to update any LES by using a CSV dump of the technologies, it crashes.

it crashes.

As in Airbrake 500 internal server error?
Maybe we should revisit the CSV import ...

Maybe we should revisit the CSV import ...

👍

Assigning @antw

If I try to update any LES by using a CSV dump of the technologies, it crashes.

I fixed that a couple of days ago in 5eb7e893 and 9715ea85.

Profiles do seem to be imported correctly from the CSV; I'll close this.


Unable to scrape socket

Hi,
I tried using this exporter and it started fine as a daemon. However, when it tried to scrape the showq scoket, it returns a permission denied error.
/var/spool/postfix/public is owned by postfix:root and showq is owned by postfix:postfix.
I have tried running the daemon as postfix user and that worked. I tried giving postfix-exporter sudoers access but that still returns the permission error to access showq.

What user are you running as? does that user have privileges? The user running postfix_exporter needs to have access.

Yeah, for some reason the permission was set really weirdly so instead of
creating a new user for postfix_exporter, we have decided to use the
postfix user to run postfix_exporter.
On Fri, Mar 27, 2020 at 11:10 PM Per Abich<EMAIL_ADDRESS>wrote:

What user are you running as? does that user have privileges? The user
running postfix_exporter needs to have access.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kumina/postfix_exporter/issues/53#issuecomment-604991789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMFSKAX3K6AXY3P37XMB3LRJSQTZANCNFSM4LT7VS6A
.

I am also having the same problem.
Postfix_exporter service is created with root user and permisions on socker are:
srw-rw-rw- 1 postfix postfix 0 Mar 31  2015 /var/spool/postfix/public/showq

Did anyone find the reason behind this ?

@learning813 Have you tried with adding user running postfix exporter to postfix and postdrop groups?
usermod -a -G postfix postfix-exporter
usermod -a -G postdrop postfix-exporter

@ndelic0 : its the postfix user itself which is running postfix-exporter

check the ownership and permissions set on following dirs
/var/spool/postifx
/var/spool/postifx/public

@ndelic0 : /var/spool/postifx
root@postfix-01•> ls -ltr /var/spool/ | grep postfix lrwxrwxrwx  1 root root    9 Jan 30 03:09 postfix -> /mailsdata root@postfix-01•> ls -ltr /var/spool/postfix/ | grep public drwx--x--- 2 postfix postdrop  88 Jan 30 03:09 public
postfix directory is pointing to "/mailsdata" which is owned by root:root which is same as /var/spool/postfix
and inside it the folder public is owned by postfix:postrdop

I recommend to only add the exporter's user to the postdrop group and not the postfix group as well. It would open up a lot of access. E.g. I run the exporter as the prometheus user and do:
usermod -a -G postdrop prometheus

On a CentOS 9 the default ownership and permissions are:
drwx--x---.  2 postfix postdrop   88 Jun 14 23:47 .
srw-rw-rw-.  1 postfix postfix     0 Jun 14 23:47 showq

So you need the postdrop group to enter into /var/spool/postfix/public only, there are no other limiting permissions.
Regarding @Misosooup , not sure what is wrong there. What user does postfix run as? Why isn't /var/spool/postfix/public owned by postfix:postdrop? Are you sure you're running as the root account (it looks like you're not)?

Hi, I have the same problem when using docker-mailserver, did you find a fix for that ?


Mysterious parameters for plot-covg-hist.R script

In the workflow examples for plotting coverage distribution you give an example of running this plot with
scripts/plot-covg-hist.R covg.before_cleaning.csv covg.before_cleaning.pdf

However, when I do this I get an error saying I am missing two parameters [<cutoff> [<kcov>]]. What do these mean/do? Would be great if you could add a passage about this in the wiki.

The last two arguments were supposed to be optional but were required by a bug. They should now be optional again (git pull to get fixed version).

cutoff is the cleaning cutoff used/proposed
kcov is approximate kmer coverage of your sample (see https://github.com/mcveanlab/mccortex/wiki/Kmer-coverage)


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/TA2k/ioBroker.viessmannapi

[ ] [E605] No actual year found in copyright. Please add "Copyright (c) 2022 TA2k<EMAIL_ADDRESS>at the end of README.md
[ ] [E701] No actual year found in LICENSE. Please add "Copyright (c) 2022 TA2k<EMAIL_ADDRESS>at the start of LICENSE

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Thanks, that all bugs have been fixed.


Improve CI stability

rh-pre-commit.version: 2.0.3
rh-pre-commit.check-secrets: ENABLED

/test test-pipeline-service-deployment-ocp-414


Update README for meteor package installation

Since I'll be deprecating chipcastledotcom:jspdf-autotable from Atmosphere, I wanted to update the docs so we don't have any more users installing it.
My only concern is that installing this new package from Atmosphere shows version 1.3.0 as the latest, yet it appears from your tags that you're currently at 1.3.2:
jspdf:autotable  1.3.0  jsPDF-AutoTable (official): generate PDF tables on the client-side (jspdf:core plugin)

I'll loop in @splendido to see if there's anything we need to do to get this updating properly.
Thanks!

Sounds strange, I double checked that the webhook is still working now.

hey guys,
sorry for the late reply. Also, I'm sorry I'm not going to be able to access the web regularly for the next month.
@someatoms, the hook is properly set up, but the automation for the wrappers is still to be finished.
I kindly ask @chip to manually publish the updates until this will be finished.
Thank you for your patience!
Cheers


Configurable Keystore for Gradle Configuration Cache

Expected Behavior
The Gradle Configuration Cache should change the behavior of EncryptionService.kt to either:

Use an environment variable, perhaps GRADLE_CONFIGURATION_CACHE_KEYSTORE_PASSWORD, so that we can inject the keystore password as a build secret on CI. This approach is probably easier, but there might be security concerns about caching a keystore that can eventually be cracked. I took a stab at it here.
Use environment variables to deterministically generate the keystore based on injected build secrets on CI. It appears that the .lock file in the cc-keystore directory is required for reading the keystore and I don't understand how that works, would need a little help on this approach.

Current Behavior
The Gradle Configuration Cache currently uses EncryptionService.kt with a hard-coded password cc.
Context
The encryption was added due to the security vulnerability of exposing build secrets via caching the configuration cache, but I've been able to see very significant performance wins in both Hinge's large Android project as well as extremely small starter projects on CI, so I believe its in the interest of Gradle and the community to support a method to share the configuration cache between machines that doesn't expose build secrets.

Thank you for your interest in Gradle!
This issue will be closed as a duplicate of

#13510

Please add your use case and 👍 to that issue.

While the original issue predates the encryption feature making the cache sharable will need to account for the encryption feature.

@cobexer I agree that having a configurable keystore is necessary for making the configuration-cache shareable across different machines, the scope of this issue is much smaller and I don't think it's helpful to merge these 2 issues.
The driver for this issue is to make the configuration cache safe to store and reuse in GitHub Actions.  This is a very narrow case of "reuse across machines", since the environment is completely controlled, the machines involved are near identical, and the Gradle User Home is shared between the machines. This functionality has been implemented, but had to be disabled due to the potential to leak secrets in the configuration cache data.
Currently, there is no way to truly secure the configuration cache data without encrypting or securing access to the Gradle User Home directory. Allowing a user to specify an encryption key for the configuration cache (instead of using the built-in hardcoded one) would be relatively straightforward, and would allow us to reuse configuration cache data safely in GitHub Actions.
In summary, I'd like you to consider this as a separate feature request for improving Gradle. By making it part of the much larger request of general configuration-cache reuse, it will effectively move into the "not in the foreseeable future" bucket, which would be sad.

This issue needs a decision from the team responsible for that area. They have been informed. Response time may vary.

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team and is prioritized by them.

@mlopatkin I'd be happy to contribute to the solution. From the above proposal is there a particular direction the team wants to take?

@mlopatkin I'd be happy to contribute to the solution. From the above proposal is there a particular direction the team wants to take?

Thank you for your suggestion. We need some time to discuss the desired shape of the solution, but we'll keep you posted.

@kaeawc @bigdaz Since we are talking about using environment variables for the keystore password, or as an input for the keystore generation... some other options:

use an environment variable to provide the encryption key, and skip the use of a keystore altogether
provide an externally generated keystore placed at a custom location (provided as an option or env var) that is created ahead of calling Gradle

Did you consider any of these options? #1 was how the encryption key was configured in the first incarnation of the encryption feature. What are your thoughts, re: pros/cons?

@abstratt @bigdaz As long as we have backwards compatibillity with the existing keystores, I think either generating the keystore from env variables or using an env variable as the entire key is fine. With either route the idea would be for official Gradle CI actions/orbs/plugins to look for the existence of this env variable in deciding whether to store the configuration cache between runs.
As for your second suggestion, an externally generated keystore with environment variable path is more complex than an environment variable. I'd rather keep this change as simple as possible for people to use so that:

The official Gradle GitHub Action can start to make use of it and most OSS projects start to benefit.
Build engineers already have lots of footguns via configuration, fewer options is probably better.

I believe @bigdaz you were also in favor of just env variable as the encryption key, right?

@abstratt looks great, solves the need, thanks!

This feature will be part of the 8.6 release. Please see release notes (and related documentation) in a recent nightly:
https://docs.gradle.org/nightly/release-notes.html#other-improvements

@abstratt I am getting this issue:
* What went wrong:
[2023-12-13 10:15:31] Could not load the value of field `parameters` of `org.gradle.api.internal.provider.DefaultValueSourceProviderFactory$DefaultObtainedValue` bean found in field `obtainedValue` of `org.gradle.configurationcache.fingerprint.ConfigurationCacheFingerprint$ValueSource` bean found in Gradle runtime.
[2023-12-13 10:15:31] > Failed to instrument class org/jetbrains/kotlin/gradle/plugin/internal/CustomPropertiesFileValueSource$Parameters in ClassLoaderScopeIdentifier.Id{coreAndPlugins:settings[:module-configuration]:buildSrc[:module-configuration]:root-project[:module-configuration](export)}

I have a :module-configuration which is a normal included build, there is no buildSrc in the project, so I am not sure why it is mentioned there.

@JavierSegoviaCordoba I am confused. Is that really related to the present issue? What you describe seems to be mentioned in https://github.com/gradle/gradle/issues/25864.

@abstratt I think we are not caching all necessary files consumed by the configuration cache.
@bigdaz do you know if there is a guide about what should we cache which would be a copy of the behavior of the Gradle Build Action?
I cannot use the Gradle Build Action in one project as it is using GitLab CI/CD.

@JavierSegoviaCordoba Please open a new issue if you are seeing a problem with Gradle and provide (in that new issue) details that allow us to reproduce or at least understand what you are seeing.


Invalid results file after running ordered test

I am running ordered test  on remote machine using Visual Studio test runner in my CD(release) pipeline.  Please find the details of the version below:
Task         : Visual Studio Test
Description  : Run tests with Visual Studio test runner
Version      : 2.0.55
Author       : Microsoft Corporation
The ordered test runs with out any issue and it logs how may tests in the ordered test passed/failed etc.
However the test tab doesn't show any details of the test execution.
This is command which is getting executed:
c:\agent\TestPlatform\vstest.console.exe C:\agent_work\r1\a\SmokeTestSuite.orderedtest /logger:trx "/TestAdapterPath:"C:\agent_work\r1\a"" /platform:x64
I am seeing the below  warning in the log after the executing of the ordered test:
##[warning]Invalid results file. Make sure the result format of the file 'C:\agent_work\r1\a\TestResults\SupportUser_FP_ACCEPT01_2019-02-01_09_52_53.trx' matches 'VSTest' test results format.
I am not sure how to fix this. Any help will be appreciated.
Thanks...

@B-Raya - The thread has been inactive for more than 90 days now. We are closing this issue.
If this issue is persisting for you, please open this issue on the tasks repo for our engg. team to take a look and help with the next steps.


Added Buffers (item and power)

Description
I've been absent for a long time now but I've been following your amazing bug fixing adventures...
During this time I worked on buffers (Item + Power) which allows the player to have a 3x3 item buffer and a power buffer that allows the player to control I/O energy rates from the block.
It implements an improved version of the energy textbox widget that I showed you on PR #364 so when this gets in the code base, I will close #364.
I'm not sure about everything in my code as usual so I will be taking your suggestions into consideration :) I've commented that in the code so you will understand what I'm talking about...
Closes #150 , #364

Checklist:

[X] My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you).
[X] I have performed a self-review of my own code.
[X] I have commented my code in areas it may be challenging to understand.
[X] I have made corresponding changes to the documentation.
[X] My changes are ready for review from a contributor.

Will not be merged into dev/1.20.1 - needs rebasing on 20.4+ to for this :) (I know you're likely busy - I'm writing this for tracking purposes)


For a 0.1.3 release

#9

Codecov Report

Merging #10 (0afc74a) into master (1afefc4) will decrease coverage by 0.11%.
Report is 4 commits behind head on master.
The diff coverage is 100.00%.

: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.
@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   91.94%   91.83%   -0.11%
==========================================
  Files          14       14
  Lines         695      698       +3
==========================================
+ Hits          639      641       +2
- Misses         56       57       +1

Files Changed
Coverage Δ

src/confusion_matrices.jl
96.55% <100.00%> (+0.06%)
:arrow_up:

... and 1 file with indirect coverage changes


feat: remove "custom" prefix from hook arguments

We shouldn't be calling the arguments with a "custom" prefix. If users are defining their own arguments that's because they want to use their own values. So, one can already infer that it is "custom".

Closing... Just realized about the issue...


Fix first index of randstrobe start index to be always the 0th randstrobe

Previously 0th index was always skipped at the randstrobe start index. Also, count variable was initialized to 0, so we would end up in the ++tot_mid_ab case, even if this was a single occurrence.

Hi Marcel, thanks for the comments. I have updated the formatting accordingly.

Looks good now, thanks again!


Error in solution for "Basic JavaScript: Use the Conditional (Ternary) Operator"

The solution for "Basic Javascript: Use the Conditional (Ternary) Operator", linked below, contains a an error:
Challenge: https://learn.freecodecamp.org/javascript-algorithms-and-data-structures/basic-javascript/use-the-conditional-ternary-operator
Solution: https://guide.freecodecamp.org/certifications/javascript-algorithms-and-data-structures/basic-javascript/use-the-conditional-ternary-operator/
Indeed, the working solution is not: return (a = b ? true : false ).
It's return (a == b ? true : false ).

@erwncmps Thank you for reporting this issue found on production.  The challenge has been recently rewritten, but the changes have not been pushed to production.  You can view the new version of the challenge on our beta testing site.
That being said, I noticed the guide article on the beta has not been updated to match the change the revision of the challenge shown on the beta site above.
This means there is still an issue which needs resolution here.

For anyone who wants to correct the guide article, the Guide article file is located here.
If you have questions about how to contribute, please see How to work on Guide articles.  If you have any questions, you can visit our Contributors room on Gitter.

Thanks, Randell. Looks good to me!

I can contribute to this

@Snowcat8 Looks like @rupali317 created a PR (#35388) to solve the issue.  Feel free to look at the changes @rupali317 made and comment on that PR if you find it does not work.


Lost events due to setting EnableSourceContainerTracking

I use TraceEventSession to collect ETW events generated by EventSource running in container. The observed issue is:

PerfView will prompt lost events during opening the ETL file even if I increase all the buffer size. There is no schema for the collected ETW events.
If I removed the flag EnableSourceContainerTracking, everything works as expected.

Could you help with this issue, thanks!

We should track this in https://github.com/dotnet/runtime/issues/87255


Update http4s-circe, http4s-client, ... to 1.0.0-M22

Updates

org.http4s:http4s-circe
org.http4s:http4s-client
org.http4s:http4s-dsl

from 1.0.0-M21 to 1.0.0-M22.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-pre-release

Superseded by #204.


Function using odata parameter

Hi,
I am trying to do the following:
var response = client.For("Endpoint")
                                 .Function("Function")
                                 .Set(new { Param1= guid, Param2 = guid, DateTime = dateTimeOffset })
                                 .ExecuteAsScalarAsync<ResultObject>();

My problem is that ":" are not supported in the url on the server i get error:

A potentially dangerous Request.Path value was detected from the client (:).

On other clients the problem is solved using an odata parameter. How can i achieve this with simple odata client. So the above code will produce Uri:

http://localhost/Endpoint/EntitySets.Function(Param1%3D1026eadb-c704-47f4-b526-c147ac1bb03a%Param2 %3D493c5dc7-c1dd-447c-b683-ff81c2b6b402%2CDateTime%3D2016-03-22T13:00:00Z)

I am looking to achieve:

http://localhost/Endpoint/EntitySets.Function(Param1%3D1026eadb-c704-47f4-b526-c147ac1bb03a%Param2 %3D493c5dc7-c1dd-447c-b683-ff81c2b6b402%2CDateTime%3D@p)?@p=2016-03-22T13%3A00%3A00Z

Thanks in advance,
Mihai

Sorry for the late response. Unfortunately in the present version Simple.OData.Client doesn't syntax for specifying function/action parameters in a query string. I'll add this as a feature for the version 5.

Mihai,
I believe you need this because you run into issue where function parameter values contain some special characters like dot and colon.
After long search I understand that IIS allows these characters only if they are the part of url query parameters (after ? sign)
For example:
this doesn't work
www.somesite.com/odata/Test(X=1, Y=2.0)
but this does work
www.somesite.com/odata/Test(X=@p1, Y=@p2<EMAIL_ADDRESS>Until this is solved in next major release I suggest you use my workaround.
Update your web.config file so it contains:
<system.web>

</system.web>
// this solves problem with dots only
// But next code solves probles with colon sign ( : ) also.
<system.webServer>

</system.webServer>

Mihai,
I believe you need this because you run into issue where function parameter values contain some special characters like dot and colon.
After long search I understand that IIS allows these characters only if they are the part of url query parameters (after ? sign)
For example:
this doesn't work
www.somesite.com/odata/Test(X=1, Y=2.0)
but this does work
www.somesite.com/odata/Test(X=@p1, Y=@p2<EMAIL_ADDRESS>Until this is solved in next major release I suggest you use my workaround.
Update your web.config file so it contains:
<system.web>

</system.web>
// this solves problem with dots only
// But next code solves probles with colon sign ( : ) also.
<system.webServer>

</system.webServer>


Add MessageChannel#editMessageById(long messageId, String newContent)

Feels like this one is missing.

Pretty sure this is fine.

This has already been added in #453 and would cause conflicts. If this is an important change I'll consider merging it, otherwise I don't see why we should rush it and cause unwanted conflicts.


d2i_X509 functions unpredictible refcount decrement of passed in object in error case

This affects all ref-counted objects: X509, X509_REQ and X509_CRL
and the functions d2i_X509(), d2i_X509_ALT(), d2i_X509_REQ(), d2i_X509_CRL and
further complicates the semantic of corresponding PEM_read_ function family
similar to #22780 just with the additional problem that if a ref-counted object
is passed in with more than one reference, it is possible that the ref-count of the
object is sometimes decremented and sometimes not decremented, but the pointer
is only set to NULL if the last reference was deleted.
so most likely a double free will be the result.
The problem is that most d2i functions are implemented using this function:
https://github.com/openssl/openssl/blob/2d70cc9cecf8b322d795985efecee06242b203b3/crypto/asn1/tasn_dec.c#L114-L121
so when any error happens, the passed in object is deleted by ASN1_item_ex_free(pval, it);
But in case this is a reference counted object and this is not the last reference that is detected here:
https://github.com/openssl/openssl/blob/2d70cc9cecf8b322d795985efecee06242b203b3/crypto/asn1/tasn_fre.c#L86-L93
so the reference count is decremented but it was not the last reference, now the *pval is not set to NULL,
while in all other cases this code does set the passed in *pval to NULL:
https://github.com/openssl/openssl/blob/2d70cc9cecf8b322d795985efecee06242b203b3/crypto/asn1/tasn_fre.c#L114-L117
To further complicate the situation there are is also the function d2i_X509_AUX here:
https://github.com/openssl/openssl/blob/2d70cc9cecf8b322d795985efecee06242b203b3/crypto/x509/x_x509.c#L207-L216
where the passed in reference is not decremented in case of error, and the *pval is again not set to NULL,
so from the caller's perspective it is impossible to decide what to do now.

@bernd-edlinger any suggestions on how to fix this?

I think there are lot more problems in those code paths. just took a look at CRYPTO_THREAD_lock_free() function here:
132 void CRYPTO_THREAD_lock_free(CRYPTO_RWLOCK *lock) {
133     if (lock == NULL)
134         return;
135
136     *(unsigned int *)lock = 0;
137     OPENSSL_free(lock);
138
139     return;
140 }

looks like lines 136 and 137 are just swapped. I believe there is no point to call OPENSSL_free(NULL); I will keep poking around. But I don't know what to think about all that now.

I would set the passed in reference to NULL in tasn_fre.c in both cases,
when it was the last reference and when it was not the last reference.
I would extend #22809 to cover the case of ref-counted objects, and
expect the application to store additional references elsewhere.

I wonder if there is any meaningful use of this API that would be broken by such change. Perhaps we should fix this only in a subsequent minor release, not in stable branches?

I wonder if there is any meaningful use of this API that would be broken by such change.

I doubt that very much.
If a caller passes in a ref-counted object, like the modified test case(s) in #22809 demonstrate,
he will be surprised (new test with idx&1 failed without the fix), and encounter a
double free at the same time, unless he did not clear the references in his error handling,
which implies a memory leak in other error cases.


channel.get usage

Hello,
In issue #70 , I was advised to use channel.get to receive messages from queue if consumer is intermittent receiver.
I modified above receive.cpp code's from example directory to use channel.get feature.

But, If you run sender first(example directory send.cpp) and few messages are being sent to RabbitMQ and then you start receiver, it will get the message the first message which was sent.
Strange thing is that It keeps receiving same first message if you run this program again and again.
I tried finding channel.get working sample but couldn't find it anywhere.
I will appreciate any help.
Thanks & Regards,
Owais.
#include <boost/date_time/posix_time/posix_time.hpp>
#include "asiohandler.h"
#include "msgpackdemo.h"

int receive(void)
{
		boost::asio::io_service ioService;
		AsioHandler handler(ioService);
		AMQP::Connection connection(&handler, AMQP::Login("guest", "guest"), "/");
		AMQP::Channel channel(&connection);
		handler.connect("localhost", 5672);
		channel.declareQueue("hello");

		channel.get("hello").onSuccess([](const AMQP::Message &message, uint64_t deliveryTag, bool redelivered) {

		std::cout << "Message fetched" << std::endl;
		std::string ss  = message.message();
		std::cout <<" [x] Received["<< std::endl;
		cout << ss << endl;
		std::cout <<"]"<< std::endl;

	   }).onEmpty([]() {

		   std::cout << "Queue is empty" << std::endl;

	   });

	ioService.run();
}

int main(void)
{
	std::cout << " [*] Waiting for messages. To exit press CTRL-C\n";
		receive();
			cout << endl << "Here-1" << endl;
		receive();
			cout << endl << "Here-2" << endl;
		receive();
			cout << endl << "Here-3" << endl;
	return 0;
}

Don't forget to acknowledge your messages using channel.ack(deilveryTag). If you do not ack a message, it is put back in the queue. This prevents that messages ever get lost when your client suddenly crashes.

Thanks for reply,
channel.ack(deilveryTag)  usage got the message removed from queue.
I got few more questions.

If there is no message in the queue when receiver is run, It enters into channel.get.onEmpty function and never receives anything again until program is restarted.

If there is a message in the queue when receiver is run, It enters into channel.get.onSuccess function and received the message and never receives anything again until program is restarted.

If i break the ioService.run by caling ioService.stop in channel.get.onSuccess or channel.get.onEmpty and try to run it again by calling ioService.run it never works.

Thanks & Regards,
Owais.

I just observed a problem, you can't specify "tag" in get like you do in "consume" function :|
I think I should revert to consume method and try to use pause and resume? What do you advice.
Thanks & Regards,
Owais.

I tried calling channel.pause().onSuccess from  channel.consume("hello", AMQP::noack).onReceived and received below error.
AMQP error NOT_IMPLEMENTED - active=false
Error reading:system:104
I googled and found it means method is not implemented.

Well that's clear, RabbitMQ does not implement this feature. I'll check if this is on the roadmap to be added and if not we'll remove this method.
The only alternative is to not pause but to simply cancel the consumer. This is basicaly what the .get() method does internally.


Subobjects being given _type in types when no _type exists in data

Illustration is given a _type: "illustration" although it's not a type.
Schema:
export default {
  title: 'USP',
  name: 'usp',
  type: 'object',
  fields: [
    {
      title: 'Illustration',
      name: 'illustration',
      type: 'object',
      fields: [
        {
          title: 'Type',
          name: 'type',
          type: 'string',
          options: {
            list: ['circle', 'lines'],
          },
        },
        {
          title: 'Colour',
          name: 'colour',
          type: 'string',
          options: {
            list: ['coral', 'yellow', 'green'],
          },
        },
      ],
    },
  ]
}

Data:
{
  "_key": "*****",
  "_type": "usp",
  "illustration": {
    "colour": "green",
    "type": "circle"
  },
}

Outputted type:
export type SanityUspType = {
  _type: 'usp';
  /**
   * Heading — `array`
   *
   *
   */
  heading?: Array<SanityKeyed<SanityBlock>>;

  /**
   * Background colour — `string`
   *
   *
   */
  backgroundColour?: 'blue-tint' | 'coral-tint' | 'green-tint';

  /**
   * Illustration — `object`
   *
   *
   */
  illustration?: {
    _type: 'illustration';
    /**
     * Type — `string`
     *
     *
     */
    type?: 'circle' | 'lines';

    /**
     * Colour — `string`
     *
     *
     */
    colour?: 'coral' | 'yellow' | 'green';
  };
}

Ah, that's interesting. I would've thought the type would've been there for this scenario. Do you know the rule to when _type is applied to the object?
This one might take me the weekend to fix. Thanks for reporting it!

I'm pretty sure it's when you inline an object. So a singleton object defined, not when it's a list of objects.
We use it for data / presentation. Where we could also link it to an external object and then that would have a _type.

Something like this appears to work for me:
https://github.com/ricokahler/sanity-codegen/pull/86

I believe I've fixed this in the alpha branch. There are a few breaking changes there but if you end up previewing it, let me know how it goes!


Quarantine Administrator does not exists in Exo

Quarantine Administrator role group does not exists in Exchange Online, that a SCC role, the required role for ExO is Transport Hygiene

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks


add dot.finance

Hi! What is your PR about?

[ ] Add or edit a skin
[x] Add custom domain for your space
[ ] Add alias for migrate your space
~Add images for your space~ (deprecated) Add the image URL on the space settings directly
~Edit your space with GitHub?~ (deprecated) Please migrate your space to ENS https://docs.snapshot.page/spaces/migrate
~Create your space with GitHub~ (deprecated) Create your space with ENS instead https://docs.snapshot.page/spaces/create

Hi can you check now please? I believe it there now

There is a conflict in your PR since domains.json file has been updated, can you resolve it?
To prevent conflicts, it is recommended to add your subdomain between two existing domains rather than at the end or beginning of the list.

done


show error message and backtrace

Solve https://github.com/deis/fluentd/issues/56

Thanks for the contribution! Please ensure your commits follow our style guide. This code will be tested once a Deis maintainer reviews it.


Skipping Package from shareAll not Working

Hi
In our team we try to use the native federation for microfrontend architecture. I have tried to set up a Microfront which worked really smooth (THANK YOU for the great work) but when I want to skip a Package from the shareAll it doesn't seem to work right.
We are using the Apache Echarts library with the ngx-echarts Angular directive.
These are my dependencies in package.json:
    "@angular-architects/native-federation": "^18.0.2",
    "@angular/animations": "^18.0.5",
    "@angular/cdk": "^18.0.5",
    "@angular/common": "^18.0.5",
    "@angular/compiler": "^18.0.5",
    "@angular/core": "^18.0.5",
    "@angular/forms": "^18.0.5",
    "@angular/material": "^18.0.5",
    "@angular/material-luxon-adapter": "^18.0.5",
    "@angular/platform-browser": "^18.0.0",
    "@angular/platform-browser-dynamic": "^18.0.5",
    "@angular/router": "^18.0.5",
    "@fontsource/inter": "^5.0.18",
    "@types/luxon": "^3.4.2",
    "angular-gridster2": "^18.0.1",
    "echarts": "^5.5.1",
    "es-module-shims": "^1.5.12",
    "luxon": "^3.4.4",
    "material-symbols": "^0.20.0",
    "ngx-echarts": "^18.0.0",
    "rxjs": "~7.8.0",
    "tslib": "^2.3.0",
    "zone.js": "~0.14.3"

My federation.config.js looks like following:
const { withNativeFederation, shareAll } = require('@angular-architects/native-federation/config');

module.exports = withNativeFederation({

  name: 'mfe1',

  exposes: {
    './routes': './src/app/app.routes.ts',
  },

  shared: {
    ...shareAll({ singleton: true, strictVersion: true, requiredVersion: 'auto' }),
  },

  skip: [
    'rxjs/ajax',
    'rxjs/fetch',
    'rxjs/testing',
    'rxjs/webSocket',
    'echarts',
    // Add further packages you don't need at runtime
  ]

  // Please read our FAQ about sharing libs:
  // https://shorturl.at/jmzH0

});

And I get the following error when I try to serve the application:
 INFO  Building federation artefacts
 INFO  Preparing shared npm packages
 NOTE  This only needs to be done once, as results are cached
 NOTE  Skip packages you don't want to share in your federation config
✘ [ERROR] File 'node_modules\echarts\types\dist\charts.d.ts' is missing from the TypeScript compilation. [plugin angular-compiler]

  Ensure the file is part of the TypeScript program via the 'files' or 'include' property.

1 of 25 errors shown (disable the message limit with --log-limit=0)
 ERR!  Error bundling shared npm package
 ERR!  Build failed with 25 errors:
 ERR!  error: File 'node_modules\echarts\types\dist\charts.d.ts' is missing from the TypeScript compilation.
 ERR!  error: File 'node_modules\echarts\types\dist\renderers.d.ts' is missing from the TypeScript compilation.
 ERR!  error: File 'node_modules\echarts\types\dist\components.d.ts' is missing from the TypeScript compilation.
 ERR!  error: File 'node_modules\echarts\types\dist\core.d.ts' is missing from the TypeScript compilation.
 ERR!  error: File 'node_modules\echarts\types\dist\echarts.d.ts' is missing from the TypeScript compilation.
 ERR!  ...
 ERR!  For more information, run in verbose mode
 NOTE
 NOTE
 NOTE  ** Important Information: ***
 NOTE  The error message above shows an issue with bundling a node_module.
 NOTE  In most cases this is because you (indirectly) shared a Node.js package,
 NOTE  while Native Federation builds for the browser.
 NOTE  You can move such packages into devDependencies or skip them in your federation.config.js.
 NOTE
 NOTE  More Details: https://bit.ly/nf-issue
 NOTE
 NOTE
Initial chunk files | Names         |  Raw size
polyfills.js        | polyfills     | 186.75 kB |
styles.css          | styles        |  93.62 kB |
main.js             | main          | 251 bytes |

                    | Initial total | 280.62 kB

Lazy chunk files    | Names         |  Raw size
chunk-DBKROM7W.js   | bootstrap     |   4.91 kB |

Application bundle generation complete. [3.666 seconds]

Watch mode enabled. Watching for file changes...
NOTE: Raw file sizes do not reflect development server per-request transformations.
  ➜  Local:   http://localhost:4201/
  ➜  press h + enter to show help

When I remove the shareAll everything works right.
Can you please provide a solution or config how to fix this issue?
Thanks in advance

I found a temporary solution:
Switching version of Apache Echarts from ^5.5.1 to fixed 5.5.0 fixes the bug. Seems to be a specific Problem with the patch,
Hopefully there will be a newer compatible version soon.

I found a temporary solution:
Switching version of Apache Echarts from ^5.5.1 to fixed 5.5.0 fixes the bug. Seems to be a specific Problem with the patch,
Hopefully there will be a newer compatible version soon.

Yes, there is a problem with some libraries, solve it by using share and adding the necessary libraries to be shared instead of using shareAll.

The reason is that NF creates a bundle per entrypoint and this very package seems to have several entry points.
You could exclude everything starting with echarts/ by using a RegExp:
skip: [
/^echarts//
]


docs: reformat pushsecrets documentation to be a list

Problem Statement
I found this part of the documentation extremly hard to read and understand, so I did my best in fixing it.
Checklist

[x] I have read the contribution guidelines
[x] All commits are signed with git commit --signoff
[x] My changes have reasonable test coverage
[x] All tests pass with make test
[x] I ensured my PR is ready for review with make reviewable

It looks like mkdocs does not support indentation on code blocks within lists. This would only be achievable with a different extension. I decided to not over complicate the change, and used sections instead.
Previously:

With Change

Nice! Thanks for fixing that! :)


feat: Support vendor form fields and Vue components

Description
This is an exploratory PR to add the ability to publish custom form fields and components from composer packages.
It's a bit rudimentary... Here's what's currently possible:

publish form field partials (supports view namespacing)
publish Vue components (no namespacing, all components go to resources/assets/vendor/js/components/)

Some ideas for improvements:

[x] Publish block and repeater Blade templates
[ ] Publish block Vue components
[ ] Publish other types of assets (styles, icons)
[ ] Proper namespacing of vendor assets

Example repository
https://github.com/pboivin/twill-numeric-input
A simple form field that wraps the vue-numeric package to provide a numeric input field with a few formatting options.

Installation
Add to composer.json:
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/pboivin/twill-numeric-input"
        }
    ],

Run:
npm install vue-numeric --save

composer require pboivin/twill-numeric-input

php artisan vendor:publish --provider='PBoivin\TwillNumericInput\ServiceProvider' --tag=components

php artisan twill:build

Usage
    @formField('twill-numeric-input::field', [
        'name' => 'price',
        'label' => 'Price',
        'thousandSeparator' => ',',
    ])

Related Issues
—

Still experimental. https://github.com/area17/twill/pull/1306/commits/127aef117b58373aabae47d7cd9a3212900992cf adds a base PackageServiceProvider class that can be used to register custom components, blocks and repeaters from packages. An example from the demo package :
use A17\Twill\PackageServiceProvider;

class ServiceProvider extends PackageServiceProvider
{
    public function register()
    {
        $this->registerVueComponentsDirectory(__DIR__ . '/../resources/assets/js/components');

        $this->registerBlocksDirectory(__DIR__ . '/../resources/views/admin/blocks');

        $this->registerRepeatersDirectory(__DIR__ . '/../resources/views/admin/repeaters');

        $this->loadViews();
    }

    public function boot()
    {
    }

    private function loadViews()
    {
        $path = __DIR__ . '/../resources/views';

        $this->loadViewsFrom($path, 'twill-numeric-input');

        $this->publishes(
            [$path => resource_path('views/vendor/twill-numeric-input')],
            'views'
        );
    }
}

Just like vendor Vue components, namespacing is not supported yet. The only way to avoid clashes currently in with file name prefixes, e.g. :

MyPackageField.vue
my-package-block.blade.php
my-package-repeater.blade.php

For vendor Vue components, I think namespacing could be implemented via subfolders. This would probably require some refactoring of the current build configuration.
For vendor blocks and repeaters, namespacing could be implemented to follow the generic Blade convention of vendor-name::block-name. This would require some considerable refactoring of the BlockCollection and related classes.
Let me know what you think! Is this going on the right tracks?

@haringsrob can we port some of this work over to the new PR for Twill composer packages support?

Adding this to https://github.com/area17/twill/pull/1446


std::begin and std::end usage without specifying std namespace

What is the issue you have?
Name clash with "end" symbol

Working with an existing code base, where an extern int end; is declared in one of header files causes ambiguity.
Please describe the steps to reproduce the issue.
int end;
#include <nlohmann/json.hpp>
int main() { return 0; }

What is the expected behavior?
That library uses std::begin/std::end and avoids the ambiguity with other symbols from global namespace.
The issue seem to only appear in line 5094 (json.hpp, single header):
auto input_adapter(const ContainerType& container) -> decltype(input_adapter(begin(container), end(container)))

Following change seems to fix the issue:
auto input_adapter(const ContainerType& container) -> decltype(input_adapter(std::begin(container), std::end(container)))

Which compiler and operating system are you using?

Compiler: g++ (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
Operating system: Ubuntu 18.04

Which version of the library did you use?

latest release version 3.9.1

Also https://github.com/nlohmann/json/issues/2248


"Empty" terminal window when restoring with fish and bash open side by side

Testing #158831
Note: I don't believe this is fish shell specific, but tagging as I was testing that test plan item.

Repro:
Bash shell side by side with Fish shell in the terminal window.
Close vscode-insiders
Reopen vscode-insiders
Fish shell is there but the window on the left that was the bash window is now some sort of empty terminal window that I can't type into instead of a bash window.

As can be seen from the fish history I was configuring the fish shell for terminal integration at the time of the shutdown / restore (needed to change code to code-insiders in the fish.config).

works for me
https://github.com/microsoft/vscode/assets/29464607/c781cd51-39f8-48dd-9a33-375f3d38a01d


Attribute Authorize didn't work with simple Requirement Policy

Hi,
I have strange issue, I configured a simple policy:
services.AddAuthorization(options =>
{
	options.AddPolicy("DavidUser", policy => policy.Requirements.Add(new DavidUserRequirement()));
});

The requirement is just empty.
The Handler does just Success().
The strange thing is, if I decorate my controller with it, I just see an empty white page and the handler code (Breakpoint) is not executed.
If I remove the attribute and call:
await _authz.AuthorizeAsync(User, "DavidUser")

inside the action the handler is called and everything works.
Ideas?

Trace the traffic either with fiddler, or your browser's dev tools, or add in the status code error middleware. What status code are you seeing?
Are you sure you have an actual user? How are you authenticating?

Traced with Fiddler, and it just returns 401 and blank page.
But without entering my Requirement. (What it does if I use "_authz.AuthorizeAsync")

Can be closed, I followed a different tutorial and I think the issue was that I have to use HttpContext.Authentication.SignInAsync at some point of time to work with Requirements.
I thought that this would be just needed if I use the ASP Identity system.


Updated dependency versions.

Hiya, I've bumped some dependency versions. It compiles, but I haven't got Wayland to test that it works at runtime.
p/s: if it does work, could you please do a release as well? Thank you!

By the way, if you're open to myself becoming a maintainer, I'm happy to take this.

@azriel91 At present I'm pretty busy with school so I'd love for someone to keep maintenance actually :)

@azriel91 if you can keep the package up to date and working correctly I'll be happy to release the changes once I've tested them.

Cool, so the test example compiles, but my system doesn't run wayland. Could you check if this works as expected:
git remote add azriel91 git@github.com:azriel91/andrew.git
git fetch azriel91 maintenance/update-dependencies
git checkout maintenance/update-dependencies
cargo run --example test

window.new_seat(..) isn't called anymore as according to the smithay-client-toolkit changelog it's automatically tracked.

@azriel91 seems to be working well for me. When you ready we can merge this and then release, thanks for your work :)

Sweet thanks :v:!


Laravel 8.63.0 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v8.63.0 branch.
Before merging, you need to:

Checkout the shift-ci-v8.63.0 branch
Review all pull request comments for additional changes
Run composer update
Test your application (no tests?)

:warning: You are running version 1.2.3 of davaxi/sparkline. The latest version is 2.0.0. You should review this dependency to see if you can upgrade to the latest version.

:warning: The Shift build servers run PHP 8. If you are still running PHP 7.4 or lower you should run composer update locally to ensure you have the proper versions of your dependencies to reflect your PHP 7 environment.

:information_source: Shift detected your application only has the default tests. This Shift is designed to be used within your CI process. Tests will help you confirm these updates do not break your application.
You can get started adding tests by using the Test Generator to automatically create the model factories, test classes, and sample tests cases for your Laravel application.


Trident, Blowpipe and Serpentine Helm Charges

Is it possible that we could add a display how many charges left in Trident of the Swamp or Blowpipe at the Combat Option a.k.a combat style tab, because it kinda hassle to right click and check (I'm lazy)
For Serpentine Helm, can it be displayed on the Equipment Menu? 😄
Hope this feature goes in the client 😃

Seems like this was already in the works, but, utilizes a very intricate form of tracking.. Not to sure if this would be pushed through.. #2990

duplicate of #953


Travis update

@clement-leprovost Whoops, it seems I didn't push to master the latest release. Good catch!
I'm closing this PR, pushing these changes to master (without the typo), and reopening a PR for the Travis update 🙂


Rebuild topology when back-off happens

Connect to bootnodes if there are not good nodes from the addressbook (spec in the linked epic + meeting from yesterday).
Rebuild topology when back-off happens (spec in the linked task)

The implementation in connectivity driver is not fully optimized, and does not cover all edge cases like some routines lingering for some time. since this is not a production level component, I think this is probably enough.  The functional edge cases should be covered, though, so it should be good enough for test envs. This should be implemented better with more advanced driver.
TODO: I fixed a little bug of handling DisconnectError in libp2p here. As it is unrelated I would add unit test for this in the separate PR.

Fixed. Thanks @acud, really good catch!

@Eknir Why did you close this? :)

@janos No problem, thanks! That was a nice review!


Define new auth Strategy to comply with oidc 0.1.1

Description
Should close #613.
How to use the new class:
The passportConfigurator class allows specifying a module which to pull the AuthStrategy from. This newly defined class can be set in the providers.json file, in the field module. This also enables every RI to decide if to use the fix or not.
Please note: the path to module in the providers.json file must be absolute, as it is later loaded from a node_module.
See here an example of providers.json file, using this class:

Motivation
It implements solution 3 of #613
Fixes:

Items added

Changes:

server/boot/oidcCompatibleStrategy.js --> defines a new strategy subclassing the existing OIDC strategy and fixing the order of the input params.

Tests included/Docs Updated?

[x] Included for each change/fix?
[x] Passing? (Merge will not be approved unless this is checked)
[ ] Docs updated?
[ ] New packages used/requires npm install?
[x] Toggle added for new features?

@nitrosx fine if I merge it?

Merging is fine for me

@minottic : would you like me to merge this PR?

@minottic would you mind sharing your code snippet for the loginCallback? I'm stuck at authenticating users but getting a Unhandled error for request GET /api/v3/Users/62fb4944304d14f82fc002cb: Error: Authorization Required Error. I suspect it's because I don't have a callback function refined?


ebpf: use dedicated bindata file

run skydive-scale-tests

run skydive-scale-tests

run skydive-scale-tests


fix LICENSE file (add GPL)

the License.txt implies that the entire project is under the WTFPL.
However, both [fdn.rev~] and [giga.rec~] are released under the GPL, which is more restrictive than the WTFPL.
Please mention this in the License.txt

i noticed that the README mentions the differing licenses.
i'd still like to suggest to add that information to the License.txt as well.

i noticed that the README mentions the differing licenses.
i'd still like to suggest to add that information to the License.txt as well.

yup, and both these objects have the license information in it.
Well, I have to say I have plans to better document these different licenses, but the milestone for that is when I reach the first official/stable release. I want to have maybe in the readme which objects take which licenses.
Now, please, tell me exactly what's the suggestion, just add the different license texts into License.txt? Add different License files?
cheers

the suggestion is to add a short paragraph for each deviant "object" to the License.txt.
similar to https://git.iem.at/pd/Gem/blob/master/COPYING.txt#L37-48

some more observations:
there's are some media-files included with else that do not originate in this library (e.g. were created by other people, under whatever license).
i've identified at least the following:

Help-files/voice.wav
Help-files/pdlogo.gif
Help-files/tklogo.git

i have done no checks about the other WAV files in the Help-files/ folder, so I cannot make any statement about those files.

and of course the leftovers from cyclone:
shared/file.*
shared/grow.*
shared/mifi.*

i have done no checks about the other WAV files in the Help-files/ folder, so I cannot make any statement about those files.

I see, thanks for checking this all out. I actually don't even remember where I got those from... :/ - I know of another voice wavefile that's a sample (or "quote") stolen from a Monty Python sketch... so, hmm, probably not a good idea, right?
I got some stuff from cyclone and these objects have a very modest remark about it, like  [else/midi] which says it's based on cyclone's [seq] (and used the mifi dependency). But I know I have to make this better and it seems this is getting really overdue.
thanks again

@porres does having the 1.0 Final Release label on this ticket from June 2019 still make sense? Is this still an important ticket to have open?

Ok, let me do this now and sorry for the delay...
I would still like a little bit of help, since I'm really ignorant
I will check throughly, but from the top of my head, I'm also using GNU/GPL, MIT, BSD and that's
so I just copy the text of each license one below the other and that's it?

Other than that, there's;

beat~ (based on aubio)
sfz~ (based on sfizz)
sfont~ (based on fluidsynth)
plaits~ (based on Mutable Instruments' PLAITS)
free.rev~ (based on FREEVERB)
giga.rev~  (based on GIGAREV)
osc.format/osc.parse/osc.route (based on Martin Peach's stuff and 'OSC' itself)

there are multiple ways of doing this.
the simplest one is probably to just state in the LICENSE.txt that all files are are © @orres and licensed under the WTFPL unless stated otherwise within the files, and then make sure that all files that are not covered by your copyright/license have the appropriate boilerplates.
for non-text files (where it is hard to add a boilerplate), add a separate paragraph in your LICENSE.txt listing the files with  copyright holders and the resp. licenses (of course it makes sense to group files that have the same copyright holders and license).
sidenote: the LICENSE.txt currently says that the copyright holder is sam hocevar. this is most likely just a copy-and-paste error. it would be best to fix the copyright holder, so that sam does not get bugreports (and license questions) for a project they never even heard of.

currently says that the copyright holder is sam hocevar. this is most likely just a copy-and paste error.

that's how little I know about licensing (shame on me). I now see I have to put my name, but I really thouhgt that when the GNU license is like " Copyright (C) 2007 Free Software Foundation, Inc. http://fsf.org/" that you had to keep it cause they owned the license (laughing already?)
And I am now thinking how to deal with these things. An external like sfont~ should have my copyright name, cause I did the external... and the license must be GNU then... I thought I had to put the copyright to fuildsynth, but no... I just have to keep the license because it is based on it and uses mostly its code, right?
And I should do the same for an object like sfz~, putting my name as the author of the object and its license (BSD-2 clause), but then I also keep a copy of the SFIZZ code and inside that folder there is its original license with the original SFIZZ authors... right?
Now, I also checked the source code of bl.saw~ and it says
"
// License: BSD 2 Clause
// Created by Timothy Schoen and Porres, based on LabSound polyBLEP oscillators
"
sorry I bothered you @timothyschoen before just having a look...
I'm still a bit confused how to deal with it all. I'm not the single author of all objects in ELSE, I can say that copyright is mine and finishing with "and others"... and leave it to the source code to specify more about authorship (which already it does)
I'm about to finish the release anyway before I actually sort it out 100%, cause I have make it available for my students already... but I'll keep working on it and keeping this ooened.
Seems like a good thing to have a subfolder named license with copies of different licenses in different files, and one file that lists which object diverges from the main license, hopefully that works.
Also, can github show multiple libraries listed?

that's how little I know about licensing (shame on me). I now see I have to put my name, but I really thouhgt that when the GNU license is like " Copyright (C) 2007 Free Software Foundation, Inc. http://fsf.org/" that you had to keep it cause they owned the license (laughing already?)

to be honest i checked other projects that use the WTFPL how they deal with this before i commented :-) (and afaict, sam hocevar is the author of the WTFPL)
sometimes it is confusing.

an external like sfont~ should have my copyright name, cause I did the external...

well if anybody else contributed non-trivial portions of the sfont~ code (e.g they fixed a bug in your external; or they wrote the original code that you build the external on; or they wrote another external where you pilfered code from,...), then they are also copyright holders (and have to agree to the license under which the given file is published; if they contributed to your code after the fact (as in: submitted pull-requests to your repo), it is fair to assume that they published the code under the license you used at the time of their contibution; if they have written original code that you took, then of course their original license applies to their code)

but then I also keep a copy of the SFIZZ code and inside that folder there is its original license with the original SFIZZ authors... right?

yes.
a single file can have multiple copyright holders (see above).
it can also have multiple separate licenses (if the code portions are clearly separated), though that way usually lies madness. (it's usually easier to keep the original license)

Seems like a good thing to have a subfolder named license with copies of different licenses

that's probably overkill.
how many licenses are there altogether?
for well-known licenses i think it is enough to just use there well-known names and add a link to their website (or use the SPDX identifier). obviously for very short licenses like the BSD & MIT licenses, including them all in a single document works as well.

Also, can github show multiple libraries listed?

don't know what this means.

to be honest i checked other projects that use the WTFPL how they deal with this before i commented :-) (and afaict, sam hocevar is the author of the WTFPL) sometimes it is confusing.

lol great! now you had him commit a crime. (just kidding) porres is not the owner of the wftpl. this is the text of the license, the project itself is licensed under it, but this is just its text. you shouldn't put your name in it.
when you go to the faust license you see that the copyright notice is at the tope (the part saying how the project is licensed and who owns it, that's where porres' name should go)
but then you find the license texts.. like gpl which is owned by the fsf (not porres)
https://github.com/grame-cncm/faust/blob/master-dev/COPYING.txt

the whole point was to just relax about all the legal mumbo-jumbo btw
http://www.wtfpl.net/faq/
but i really wanna see a court case where: "your honor, my project was licensed with correct notice under the terms of WTFPL"

thanks for the clarifications

what's the state of this?

I have added for a while a folder with separate files for each license and a more detailed list of which object is under which license, but I kept this opened because I guess I wanted to revise this and re-check things

ah well, i wouldn't have looked into that folder, obviously just checking the License.txt file.
i noticed that the README mentions the different licenses (which is great).
i was going to complain that the README should then also mention the LICENSE/ folder to check for the license texts, but hav enow discovered, that this is also mentioned (so i missed it).
so here's a few more nitpicks/suggestions:

the README mentions a folder named license but the folder is really named LICENSE (upper-case). there are case-sensitive filesystems out there, so the documentation should probably be accurate.
the folder is called LICENSE but should probably be called LICENSES (plural)
i would welcome it, if the various license files would not have spaces in their filenames  (as license files are often referenced in machine-readable metadata, and spaces always make problems) and use consistent spelling (e.g. LICENSE vs License). probably just strip the LICENSE  part altogether and go with the license shortname (e.g. just MIT)
you probably could use the full short SPDX license specifier instead of your own abbreviations. at least three of your licenses (what you refer to as "BSD", "CC" and "GNU") have multiple versions, and it would be nice to be able to tell which one you really mean (e.g. CC-BY-NC-ND (which btw is considered a "non-free" license by many) is quite a different license from CC0 (aka "public domain")
so i would suggest the following filenames names:

BSD-2
CC-BY-NC-ND
GPL-3
MIT
WTFPL


feat: indexer cells filter

Description
This pr implements an cells filter protocol identical to ckb-indexer filter: https://github.com/nervosnetwork/ckb/blob/develop/rpc/README.md#method-get_cells
This pr resolve a task in #501
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactor (non-breaking change)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation or Website
[ ] Example
[ ] Other

How Has This Been Tested?

[x] Unit Tests

https://github.com/ckb-js/lumos/blob/0c5ea9feea5acf0e3dfa48b58d6a2a9de2b0fa90/packages/ckb-indexer/src/collector.ts#L55-L172
please refactor to the new convert to improve the test coverage

https://github.com/ckb-js/lumos/blob/0c5ea9feea5acf0e3dfa48b58d6a2a9de2b0fa90/packages/ckb-indexer/src/collector.ts#L55-L172
please refactor to the new convert to improve the test coverage

The code you referred does not convert QueryOptions to SearchKey, the original convert function is here: https://github.com/ckb-js/lumos/blob/develop/packages/ckb-indexer/src/services.ts#L18 , which is not compatible with new convert funciton, its parameter type is CKBIndexerQueryOptions

Do we need to put CKBIndexerQueryOptions in @ckb-lumos/base, and enhance new convertor to accept CKBIndexerQueryOptions as input parameter type?

I guess it would be better to make progressive changes, so just keep implementing it in @ckb-lumos/ckb-indexer. If we base on the idea of progressive changes, can we

extend the matcher so that it supports CkbIndexOptions

declare function isMatchQueryOption(cell: Cell, options:QueryOption);

function isMatchCkbIndexerQueryOption(cell: Cell, options: CkbIndexerQueryOptions) {
  if (!filterByQueryOptions(options)) return false;
  // continue to implement another filter
}

only a match function to support CkbIndexerQueryOptions in one step

declare functoin isMatchCkbIndexerQueryOption(cell, options: CkbIndexerQueryOptions): boolean;

But I haven't considered which one is more appropriate, or is there a more appropriate way?

More importantly, it occurred to me that we implemented the match(or filter) to provide a more reliable TransactionManager or enhance the current TransactionManager, in order to keep the behavior in the dApp as consistent as possible with the lumos's CkbIndexer, and to filter pending cells intuitively.
Therefore, I think we can consider how to enhance the TranasctionManager or design a new TranasctionManager before deciding how to implement the match

The interface of NewTransactionManager has been discussd already:
const manager = new TransactionManager("https://testnet.ckb.dev");

const cellCollector = manager.collector({
  lock: {
    codeHassh: "0x...",
    hashType: "type",
    args: "0x",
  },
});

for await (const cell of cellCollector.collect()) {
  console.log(cell);
}

so the cells will be filtered when cellCollector collects cells, maybe option 2 is ok

@zhangyouxin Can you revert the merge of https://github.com/ckb-js/lumos/pull/503, because the client side filter and TransactionManager can be two features. Only the #502 will be appended to changelog if we put #502 and #503 in one PR. I suggest we can merge client side filter, and open another TransactoinManager PR to target to develop

@zhangyouxin Can you revert the merge of #503, because the client side filter and TransactionManager can be two features. Only the #502 will be appended to changelog if we put #502 and #503 in one PR. I suggest we can merge client side filter, and open another TransactoinManager PR to target to develop

OK, reseted, #503 is based on #502, so we can merge #502 first


Centered the video elements with tp-video class.

Related Issues
Fixes #616
Updates

Added "tp-video" class to all the video elements
Added margin: auto; display: block to the "tp-video".

Screenshot

@Satoshi-Sh Thank you for your contributionS to improving Taipy.